Opened 10 years ago

Closed 10 years ago

#14182 closed defect (fixed)

Fix whitespace in coercion_and_categories.rst

Reported by: Jeroen Demeyer Owned by: Minh Van Nguyen
Priority: blocker Milestone: sage-5.8
Component: documentation Keywords:
Cc: Simon King Merged in: sage-5.8.beta2
Authors: Jeroen Demeyer Reviewers: Simon King
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #11490 Stopgaps:

Status badges

Description

devel/sage/doc/en/thematic_tutorials/coercion_and_categories.rst mixes TABs and spaces for indentation, which is very bad style and might also lead to doctesting trouble.

Attachments (1)

14182_whitespace.patch (9.0 KB) - added by Jeroen Demeyer 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by Simon King

How to replace tabs by four blank spaces (in linux or vim)? Perhaps you can answer faster than I can look up in google...

Changed 10 years ago by Jeroen Demeyer

Attachment: 14182_whitespace.patch added

comment:2 Changed 10 years ago by Jeroen Demeyer

Authors: Jeroen Demeyer
Status: newneeds_review

comment:3 Changed 10 years ago by Simon King

OK, you were faster. I found sed 's/\\t/ /g' oldcode.py >newcode.py in google...

comment:4 Changed 10 years ago by Simon King

Reviewers: Simon King
Status: needs_reviewpositive_review

Good. After applying the patch, I could not find any tab space in the code. Since the patch does not change the actual text, since the documentation built w/o trouble, and since the result looks fine in the browser, I give it a positive review.

Last edited 10 years ago by Simon King (previous) (diff)

comment:5 Changed 10 years ago by David Roe

Dependencies: #11490

comment:6 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.8.beta2
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.