Opened 11 years ago

Closed 11 years ago

#5757 closed defect (fixed)

[with patch, positive review] change nodoctest directive

Reported by: jhpalmieri Owned by: jhpalmieri
Priority: minor Milestone: sage-3.4.1
Component: doctest coverage Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Without this patch, if the string 'nodoctest' is anywhere in the file, then the file is not doctested. This changes it to only look at for 'nodoctest' in the first 50 characters of the file.

(This is a patch to the 'scripts' repository.)

Attachments (1)

scripts-nodoctest.patch (1.1 KB) - added by jhpalmieri 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by jhpalmieri

comment:1 Changed 11 years ago by was

  • Summary changed from [with patch, needs review] change nodoctest directive to [with patch, positive review] change nodoctest directive

comment:2 Changed 11 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 3.4.1.rc3.

Cheers,

Michael

Note: See TracTickets for help on using tickets.