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)
Change History (7)
comment:1 Changed 10 years ago by
Changed 10 years ago by
| Attachment: | 14182_whitespace.patch added |
|---|
comment:2 Changed 10 years ago by
| Authors: | → Jeroen Demeyer |
|---|---|
| Status: | new → needs_review |
comment:3 Changed 10 years ago by
OK, you were faster. I found sed 's/\\t/ /g' oldcode.py >newcode.py in google...
comment:4 Changed 10 years ago by
| Reviewers: | → Simon King |
|---|---|
| Status: | needs_review → positive_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.
comment:5 Changed 10 years ago by
| Dependencies: | → #11490 |
|---|
comment:6 Changed 10 years ago by
| Merged in: | → sage-5.8.beta2 |
|---|---|
| Resolution: | → fixed |
| Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
How to replace tabs by four blank spaces (in linux or vim)? Perhaps you can answer faster than I can look up in google...