Ticket #8160 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

add 'text' option to sphinxify

Reported by: jhpalmieri Owned by: was
Priority: minor Milestone: sage-4.3.2
Component: notebook Keywords:
Cc: mpatel Work issues:
Report Upstream: N/A Reviewers: Mitesh Patel
Authors: John Palmieri Merged in: sagenb-0.7.4
Dependencies: Stopgaps:

Description (last modified by jhpalmieri) (diff)

This patch adds a 'text' option to sphinxify: use sphinxify(s, format='text') or sphinxify(s, format='html'), where format is optional with default value 'html'. The intended use is in sage.misc.sagedoc for producing docstrings from the command line. I'll create another ticket (#8161) for that.

Depends on #8102.

Attachments

trac_8160-sphinxify.patch Download (3.8 KB) - added by jhpalmieri 3 years ago.
apply to sagenb repo
trac_8160-sphinxify_text.patch Download (3.8 KB) - added by mpatel 3 years ago.
Rebased vs. #8102. Apply only this patch. sagenb repo.

Change History

Changed 3 years ago by jhpalmieri

apply to sagenb repo

comment:1 Changed 3 years ago by jhpalmieri

  • Status changed from new to needs_review

comment:2 Changed 3 years ago by jhpalmieri

  • Description modified (diff)

Changed 3 years ago by mpatel

Rebased vs. #8102. Apply only this patch. sagenb repo.

comment:3 Changed 3 years ago by mpatel

  • Status changed from needs_review to positive_review
  • Reviewers set to Mitesh Patel

I've attached a version rebased against #8102 --- it seemed a bit easier than the opposite. Positive review for the first patch, at least.

comment:4 Changed 3 years ago by jhpalmieri

  • Description modified (diff)

comment:5 Changed 3 years ago by mpatel

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sagenb-0.7.4
Note: See TracTickets for help on using tickets.