Opened 7 years ago

Closed 7 years ago

#14011 closed enhancement (fixed)

Add various new sphinx roles

Reported by: hivert Owned by: mvngu
Priority: minor Milestone: sage-5.8
Component: documentation Keywords: Sphinx roles
Cc: Merged in: sage-5.8.beta1
Authors: Florent Hivert Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #6495 Stopgaps:

Description (last modified by hivert)

I added the following roles: arxiv, oeis, doi, mathscinet.

Apply: trac_14011-Sphinx_roles-fh.patch​

Florent

Attachments (1)

trac_14011-Sphinx_roles-fh.patch​ (4.3 KB) - added by hivert 7 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 7 years ago by hivert

  • Authors set to Florent Hivert
  • Component changed from PLEASE CHANGE to documentation
  • Keywords Shinx roles added
  • Owner changed from tbd to mvngu
  • Priority changed from major to minor
  • Type changed from PLEASE CHANGE to enhancement

comment:2 Changed 7 years ago by hivert

  • Keywords Sphinx added; Shinx removed
  • Status changed from new to needs_review

comment:3 Changed 7 years ago by chapoton

Hello,

maybe these new functions should be added to the manual page ?

http://www.sagemath.org/doc/developer/sage_manuals.html#chapter-sage-manuals-links

Frederic

comment:4 Changed 7 years ago by chapoton

Hello,

I have added the doc.

Frederic

comment:5 Changed 7 years ago by hivert

  • Dependencies set to #6495

Hi Frederic,

If you are Ok with my patch, I'm ok with yours, so you can add positive review.

Florent

comment:6 Changed 7 years ago by chapoton

  • Status changed from needs_review to positive_review

ok, positive review

comment:7 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.7 to sage-5.8
  • Reviewers set to Frédéric Chapoton

comment:8 follow-up: Changed 7 years ago by jdemeyer

  • Status changed from positive_review to needs_work

The patch needs a proper commit message.

Changed 7 years ago by hivert

comment:9 in reply to: ↑ 8 Changed 7 years ago by hivert

  • Description modified (diff)
  • Status changed from needs_work to positive_review

Replying to jdemeyer:

The patch needs a proper commit message.

Oups ! Done ! And folded the two patches.

Florent

comment:10 Changed 7 years ago by chapoton

Apply trac_14011-Sphinx_roles-fh.patch

comment:11 Changed 7 years ago by jdemeyer

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