Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC Added link to raise issue and view source code #1329

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Hrittik20
Copy link
Contributor

Issue: #1301

@@ -64,3 +64,7 @@ ReST to markdown by sphinx-gallery. This currently only works for basic ReST
functionality, so try to limit use of ReST directives (e.g., `.. note::`),
internal links (e.g., `:ref:`), and other functionality that won't render well
in markdown.


.. note::
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are rst files, not the examples themselves. The issue is about adding these notes to the examples.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh, I understand. I need to add # before each line if I put the note in .py, right?



#.. note::
# |edit| `View source <https://github.com/fairlearn/fairlearn/blob/main/examples/plot_grid_search_census.py>`to the code file or `open issue <https://github.com/fairlearn/fairlearn/issues/new?>`__.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# |edit| `View source <https://github.com/fairlearn/fairlearn/blob/main/examples/plot_grid_search_census.py>`to the code file or `open issue <https://github.com/fairlearn/fairlearn/issues/new?>`__.
# |edit| `View source <https://github.com/fairlearn/fairlearn/blob/main/examples/plot_grid_search_census.py>` or `open issue <https://github.com/fairlearn/fairlearn/issues/new?>`__.

Small suggestion for this file and others.

Did you build the docs to see if it worked as intended?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way to put the note in rst or html file for the notebook examples? Putting it in .py file just makes it a part of the examples rather than making a note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants