Opened 10 years ago
Closed 10 years ago
#12678 closed enhancement (fixed)
Extract a method to get the name of the .rst file corresponding to a python module, and doctest it.
Reported by: | nthiery | Owned by: | mvngu |
---|---|---|---|
Priority: | major | Milestone: | sage-5.0 |
Component: | documentation | Keywords: | |
Cc: | Merged in: | sage-5.0.beta10 | |
Authors: | Nicolas M. Thiéry | Reviewers: | Mike Hansen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Title says it all.
I thought I needed this method, and finally did not. I put it here in case this could be useful to someone else. Feel free to review and get merged if you want to add a doctested method to Sage :-)
Attachments (1)
Change History (8)
comment:1 Changed 10 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
- Reviewers set to Mike Hansen
- Status changed from needs_review to positive_review
comment:3 follow-up: ↓ 4 Changed 10 years ago by
- Status changed from positive_review to needs_work
Not everybody installs Sage in /opt/sage-5.0.beta8
:
sage -t -force_lib devel/sage/doc/common/builder.py ********************************************************************** File "/scratch/jdemeyer/merger/sage-5.0.beta10/devel/sage-main/doc/common/builder.py", line 606: sage: builder.ReferenceBuilder("reference").auto_rest_filename("sage.combinat.partition") Expected: '/opt/sage-5.0.beta8/devel/sage/doc/en/reference/sage/combinat/partition.rst' Got: '/scratch/jdemeyer/merger/sage-5.0.beta10/devel/sage/doc/en/reference/sage/combinat/partition.rst' **********************************************************************
Changed 10 years ago by
comment:4 in reply to: ↑ 3 Changed 10 years ago by
Replying to jdemeyer:
Not everybody installs Sage in
/opt/sage-5.0.beta8
:
Oops, sorry.
Should be fixed now!
comment:5 Changed 10 years ago by
- Status changed from needs_work to needs_review
comment:6 Changed 10 years ago by
- Status changed from needs_review to positive_review
comment:7 Changed 10 years ago by
- Merged in set to sage-5.0.beta10
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Looks good to me.