Opened 5 years ago

Closed 5 years ago

#6688 closed defect (fixed)

[with patch, positive review] spell-check all modules under sage/calculus

Reported by: mvngu Owned by: tba
Priority: major Milestone: sage-4.1.1
Component: documentation Keywords:
Cc: Merged in: Sage 4.1.1
Authors: Minh Van Nguyen Reviewers: Burcin Erocal
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

As the subject says.

Attachments (1)

trac_6688-spell-check-calculus.patch (6.2 KB) - added by mvngu 5 years ago.
based on Sage 4.1.1.rc2

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 years ago by mvngu

  • Authors set to Minh Van Nguyen
  • Summary changed from spell-check all modules under sage/calculus to [with patch, needs review] spell-check all modules under sage/calculus

comment:2 follow-up: Changed 5 years ago by burcin

  • Reviewers set to Burcin Erocal
  • Summary changed from [with patch, needs review] spell-check all modules under sage/calculus to [with patch, positive review] spell-check all modules under sage/calculus

Thanks Minh. The patch looks good and harmless. :)

BTW, the 3rd hunk of the patch shows another instance of "mathematica" not capitalized correctly.

Changed 5 years ago by mvngu

based on Sage 4.1.1.rc2

comment:3 in reply to: ↑ 2 Changed 5 years ago by mvngu

Replying to burcin:

BTW, the 3rd hunk of the patch shows another instance of "mathematica" not capitalized correctly.

Done. That has been fixed in the new patch. Thank you for pointing that out.

comment:4 Changed 5 years ago by mvngu

  • Merged in set to Sage 4.1.1
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.