Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[DOC] reshuffle of contributing#28026

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
timhoffm merged 6 commits intomatplotlib:mainfromstory645:reorg-new-contrib
Apr 6, 2024

Conversation

story645
Copy link
Member

@story645story645 commentedApr 5, 2024
edited
Loading

PR summary

Attempt to address hierarchy issues from#28005 by reordering so that:

on contributing page:

  • moved the "why should I contribute" into a default open dropdown so that it can both stand out for the folks who need it and can be collapsed for folks who don't.
  • code/docs/community contributions get moved under get-started as sub headings
  • issue related blocks get shifted into their own section
  • removed double nesting on 'codespaces workflow section'
  • slight reordering of contributing section

And on the index page:

~I haven't built yet, this is very much sketchy. ~

PR checklist

@github-actionsgithub-actionsbot added the Documentation: devdocsfiles in doc/devel labelApr 5, 2024
@story645story645 changed the title[DOC] minor reshuffle of contributing[DOC] reshuffle of contributingApr 5, 2024
@@ -94,29 +99,23 @@ New contributors
:link-type: ref
:shadow: none

:octicon:`code;1em;sd-text-info`Contribute code
:octicon:`code;1em;sd-text-info`Code
Copy link
Member

Choose a reason for hiding this comment

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

I feel just the nouns are too terse. What do the mean? Read documentation, write documentation? Documentation of the project or documentation on contributing?

"Contribute code" and "Write documentation" weren't too bad. "Join the community" could be the third one. Alternatively, "Work on code", "Work on documentation", "Work with the community".

Copy link
MemberAuthor

@story645story645Apr 5, 2024
edited
Loading

Choose a reason for hiding this comment

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

updated tobuild community -stuff like citations and adding to the third party list are part of building our community/ecosystem.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

so the triplet becomes: contribute code, write documentation, build community. I'd use a different verb on code but write code and write docs is clunky in its own way.

:octicon:`issue-opened;1em;sd-text-info` :ref:`What are "good-first-issues"? <new_contributors>`

:octicon:`git-pull-request;1em;sd-text-info` :ref:`How do I claim an issue? <managing_issues_prs>`
:octicon:`git-pull-request;1em;sd-text-info` :ref:`How do I choose an issue? <managing_issues_prs>`
Copy link
Member

Choose a reason for hiding this comment

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

I find this slightly off: I believe many contributions start because people have a certain idea or problem they are interested in. I assume it's rather uncommon that people say "Oh I want to work on Matplotlib. What can I do?"

I have no good alternative suggestion yet. Just saying the current phrasing is tailored to a rather edge case.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

I get you, not sure claim is any better...it's the 'navigate the issue tracker' bullet, :hmm:

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

"how do I work on issues" maybe?

Copy link
Member

Choose a reason for hiding this comment

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

"how do I work on issues" maybe?

Yes, or "How to work on issues".

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

So I have "Work on an issue" inside contributing.rst, but I think on index we want to keep the FAQ type structure here?

@story645story645 marked this pull request as ready for reviewApril 5, 2024 21:21
@timhoffmtimhoffm added this to thev3.9.0 milestoneApr 6, 2024
@timhoffmtimhoffm merged commit6474934 intomatplotlib:mainApr 6, 2024
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull requestApr 6, 2024
rcomer added a commit that referenced this pull requestApr 6, 2024
…026-on-v3.9.xBackport PR#28026 on branch v3.9.x ([DOC] reshuffle of contributing)
@story645story645 deleted the reorg-new-contrib branchApril 7, 2024 00:47
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@timhoffmtimhoffmtimhoffm approved these changes

Assignees
No one assigned
Labels
Documentation: devdocsfiles in doc/devel
Projects
None yet
Milestone
v3.9.0
Development

Successfully merging this pull request may close these issues.

2 participants
@story645@timhoffm

[8]ページ先頭

©2009-2025 Movatter.jp