Opened 8 years ago
Closed 8 years ago
#14182 closed defect (fixed)
Fix whitespace in coercion_and_categories.rst
Reported by: | jdemeyer | Owned by: | mvngu |
---|---|---|---|
Priority: | blocker | Milestone: | sage-5.8 |
Component: | documentation | Keywords: | |
Cc: | SimonKing | Merged in: | sage-5.8.beta2 |
Authors: | Jeroen Demeyer | Reviewers: | Simon King |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #11490 | Stopgaps: |
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 8 years ago by
Changed 8 years ago by
comment:2 Changed 8 years ago by
- Status changed from new to needs_review
comment:3 Changed 8 years ago by
OK, you were faster. I found sed 's/\\t/ /g' oldcode.py >newcode.py
in google...
comment:4 Changed 8 years ago by
- Reviewers set to Simon King
- Status changed from needs_review to 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 8 years ago by
- Dependencies set to #11490
comment:6 Changed 8 years ago by
- Merged in set to sage-5.8.beta2
- Resolution set to fixed
- Status changed from positive_review to 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...