Ticket #6916 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] follow-up to #6749: fix warnings when building reference manual

Reported by: mvngu Owned by: tba
Priority: major Milestone: sage-4.1.2
Component: documentation Keywords:
Cc: ncohen Work issues:
Report Upstream: Reviewers: William Stein, John Palmieri
Authors: Minh Van Nguyen Merged in: Sage 4.1.2.rc0
Dependencies: Stopgaps:

Description

As the subject says. This is a follow-up to ticket #6749.

Attachments

trac_6916-typos.patch Download (10.6 KB) - added by mvngu 4 years ago.
based on Sage 4.1.2.alpha4

Change History

comment:1 Changed 4 years ago by mvngu

  • Authors set to Minh Van Nguyen
  • Cc ncohen added
  • Component changed from algebra to documentation
  • Summary changed from follow-up to #6749: fix warnings when building reference manual to [with patch, needs review] follow-up to #6749: fix warnings when building reference manual
  • Milestone set to sage-4.1.2
  • Owner changed from tbd to tba

comment:2 Changed 4 years ago by was

  • Summary changed from [with patch, needs review] follow-up to #6749: fix warnings when building reference manual to [with patch, positive review] follow-up to #6749: fix warnings when building reference manual

comment:3 follow-up: ↓ 4 Changed 4 years ago by jhpalmieri

  • Reviewers set to William Stein, John Palmieri

I give it a positive review, too, but if you want to change the lines

    - ``seq`` -- Two different possible types: 
      - A sequence of pairs (weight, value). 
      - A sequence of reals (a value of 1 is assumed). 

by adding a blank line after the first one, it looks a little better (to me) when typeset. I'll give that a positive review in advance, so if you want to make the change, no need for more review.

Changed 4 years ago by mvngu

based on Sage 4.1.2.alpha4

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

Replying to jhpalmieri:

I give it a positive review, too, but if you want to change the lines

    - ``seq`` -- Two different possible types: 
      - A sequence of pairs (weight, value). 
      - A sequence of reals (a value of 1 is assumed). 

by adding a blank line after the first one, it looks a little better (to me) when typeset.

Done.

comment:5 Changed 4 years ago by mvngu

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