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: improve tagging guidelines page #28527

Merged
merged 3 commits into from
Jul 11, 2024
Merged

Conversation

story645
Copy link
Member

@story645 story645 commented Jul 9, 2024

Based on discussions w/ @esibing after #27235 and so it's improved for scipy sprints, cleaned up the tagging page so:

  • list of categories pulls from glossary section headings so will remain in sync
  • added a code example of how to add tags
  • move content guidelines to writing guide -> I don't want/expect it to stay there, but think it's distracting here.

Also, thanks to @melissawm and melissawm/sphinx-tags#102

  • enable multiline

PR summary

PR checklist

@story645 story645 added Documentation: devdocs files in doc/devel Documentation: tags tagging examples + proposing new tags labels Jul 9, 2024
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/_static/mpl.css Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/_static/mpl.css Outdated Show resolved Hide resolved
@story645 story645 force-pushed the tag-guide branch 2 times, most recently from 37587bf to 502fb6b Compare July 9, 2024 21:13
@story645
Copy link
Member Author

story645 commented Jul 9, 2024

Attempted to address all the comments

Copy link
Member

@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

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

Take or leave my comments. You can self-merge after that (and fixing the sphinx build).

doc/devel/document.rst Outdated Show resolved Hide resolved

All :ref:`examples-index` should aim to follow these guidelines:

:Title: Describe content in a short sentence (approx. 1-6 words). Do not use *demo* as
Copy link
Member

Choose a reason for hiding this comment

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

Does sentence imply a full sentence including verb? Possible alternatives: expression, phrase ... (not a native speaker so cannot really judge)

doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/devel/document.rst Outdated Show resolved Hide resolved
doc/_static/mpl.css Outdated Show resolved Hide resolved
doc/_static/mpl.css Outdated Show resolved Hide resolved
story645 and others added 3 commits July 11, 2024 05:12
add coding example
move content guidelines to writing guide
enable multiline

Co-authored-by: Eva Sibinga <46283995+esibinga@users.noreply.github.com>
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
@story645 story645 merged commit 48c8f07 into matplotlib:main Jul 11, 2024
44 checks passed
@story645
Copy link
Member Author

story645 commented Jul 11, 2024

Thanks, self merged after addressing the comments so this would be live for when we advertise tagging next session, but will totally make a follow up if anyone has feedback.

@QuLogic QuLogic added this to the v3.10.0 milestone Jul 12, 2024
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Jul 12, 2024
Following recommendatinos from matplotlib#28527, this improves example titles. Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Jul 27, 2024
Following recommendatinos from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a
glance when going through the examples once. Certainly, one could do
further improvements, but that can be done in follow-ups.

Co-authored-by: hannah <story645@gmail.com>
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 1, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 1, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 1, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 1, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 2, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.
timhoffm added a commit to timhoffm/matplotlib that referenced this pull request Aug 2, 2024
Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw  t a
glance when going through the examples once. Certainly, one could do
further improvements, but that can be done in follow-ups.

Co-authored-by: hannah <story645@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation: build building the docs Documentation: devdocs files in doc/devel Documentation: tags tagging examples + proposing new tags
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants