Ticket #6688 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[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: Work issues:
Report Upstream: Reviewers: Burcin Erocal
Authors: Minh Van Nguyen Merged in: Sage 4.1.1
Dependencies: Stopgaps:

Description

As the subject says.

Attachments

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

Change History

comment:1 Changed 4 years ago by mvngu

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

comment:2 follow-up: ↓ 3 Changed 4 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 4 years ago by mvngu

based on Sage 4.1.1.rc2

comment:3 in reply to: ↑ 2 Changed 4 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 4 years ago by mvngu

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