Skip to content

MAINT: Fix test for links#811

Merged
larsoner merged 2 commits intosphinx-gallery:masterfrom
larsoner:fix
Apr 13, 2021
Merged

MAINT: Fix test for links#811
larsoner merged 2 commits intosphinx-gallery:masterfrom
larsoner:fix

Conversation

@larsoner
Copy link
Copy Markdown
Contributor

Locally I have this failure, let's see if it fixes the problems in #810

@larsoner larsoner merged commit 83479d1 into sphinx-gallery:master Apr 13, 2021
@larsoner larsoner deleted the fix branch April 13, 2021 17:21
@lucyleeow
Copy link
Copy Markdown
Contributor

Thanks for fixing this @larsoner

lucyleeow pushed a commit that referenced this pull request Apr 15, 2021
* DOC: clarify advanced doc wrt referencing examples

As discussed via #805, the documentation could be structured a bit better to make it easier to find how to link examples. This PR suggests adding a new sub-heading *Referencing Examples* so that the term *reference* explicitly shows up in the table of contents. 

Two things to discuss:
* I'm not sure yet whether the heading should be *referencing* or *linking*. 
* Is it possible to link to heading in examples, too? Is there a special syntax for that? If yes, I could extend this PR to include it.

* DOC: add 2 projects using sphinx gallery (#807)

These are:
* https://github.com/automl/auto-sklearn
* https://github.com/openml/openml-python/

* MAINT: Fix test for links (#811)

* MAINT: Fix test for links

* FIX: Property

* Apply suggestions

* Update advanced.rst

* Update advanced.rst

* minor amendment

Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
Co-authored-by: Lucy Liu <lucy.liu@bom.gov.au>
@lucyleeow lucyleeow added the bug label Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants