Opened 8 years ago

Closed 8 years ago

#14362 closed enhancement (fixed)

Fix more optional tags

Reported by: jdemeyer Owned by: mvngu
Priority: minor Milestone: sage-5.9
Component: doctest coverage Keywords:
Cc: Merged in: sage-5.9.beta2
Authors: Jeroen Demeyer Reviewers: Karl-Dieter Crisman
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Found by #13278.

Attachments (1)

14362_optional.patch (25.3 KB) - added by jdemeyer 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by jdemeyer

comment:1 Changed 8 years ago by jdemeyer

  • Status changed from new to needs_review

comment:2 Changed 8 years ago by kcrisman

  • Reviewers set to Karl-Dieter Crisman
  • Status changed from needs_review to positive_review

Awesome. This looks fine. Assuming it doesn't conflict with something else related to all these doctest framework ticket, positive review - no functional change, the tests "unoptionaled" are ok.

comment:3 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.9.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.