sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.4k stars 473 forks source link

Add various new sphinx roles #14011

Closed hivert closed 11 years ago

hivert commented 11 years ago

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

Apply: attachment: trac_14011-Sphinx_roles-fh.patch

Florent

Depends on #6495

Component: documentation

Keywords: Sphinx roles

Author: Florent Hivert

Reviewer: Frédéric Chapoton

Merged: sage-5.8.beta1

Issue created by migration from https://trac.sagemath.org/ticket/14011

hivert commented 11 years ago

Changed keywords from none to Shinx roles

hivert commented 11 years ago

Author: Florent Hivert

hivert commented 11 years ago

Changed keywords from Shinx roles to Sphinx roles

fchapoton commented 11 years ago
comment:3

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

fchapoton commented 11 years ago
comment:4

Hello,

I have added the doc.

Frederic

hivert commented 11 years ago
comment:5

Hi Frederic,

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

Florent

hivert commented 11 years ago

Dependencies: #6495

fchapoton commented 11 years ago
comment:6

ok, positive review

jdemeyer commented 11 years ago

Reviewer: Frédéric Chapoton

jdemeyer commented 11 years ago
comment:8

The patch needs a proper commit message.

hivert commented 11 years ago
comment:9

Attachment: trac_14011-Sphinx_roles-fh.patch.gz

Replying to @jdemeyer:

The patch needs a proper commit message.

Oups ! Done ! And folded the two patches.

Florent

hivert commented 11 years ago

Description changed:

--- 
+++ 
@@ -1,4 +1,6 @@
 I added the following roles: arxiv, oeis, doi, mathscinet.
+
+**Apply:** [attachment: trac_14011-Sphinx_roles-fh.patch](https://github.com/sagemath/sage-prod/files/10656999/trac_14011-Sphinx_roles-fh.patch.gz)

 Florent
fchapoton commented 11 years ago
comment:10

Apply trac_14011-Sphinx_roles-fh.patch

jdemeyer commented 11 years ago

Merged: sage-5.8.beta1