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

Fix dead links in dev workflow docs#29463

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 2 commits intomatplotlib:mainfromKhushikela29:main
Jan 13, 2025
Merged

Conversation

Khushikela29
Copy link
Contributor

@Khushikela29Khushikela29 commentedJan 12, 2025
edited by timhoffm
Loading

PR summary

  • This pull request addresses the removal of a broken link in the doc/devel/development_workflow.rst and updates the content to improve clarity and accuracy.
  • The original link towhy-the-flag? was broken, and the resources are no longer available.
  • Resolving the issue by replacing "For more information, seewhy the -a flag? and thegit commit manual page." to "For more information, see thegit commit manual page."

Closes#29305.

PR checklist

@github-actionsgithub-actionsbot added Documentation: examplesfiles in galleries/examples Documentation: devdocsfiles in doc/devel labelsJan 12, 2025
Copy link
Member

@timhoffmtimhoffm left a comment

Choose a reason for hiding this comment

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

You seem to also have the commits from#29433 in this PR. While they are identical, the marge would go through, but please rebase so that it's clear these commits don't have anything to do with this PR.

@timhoffmtimhoffm changed the titleFixes #29305Fix dead links in dev workflow docsJan 12, 2025
@timhoffm
Copy link
Member

On a side-note, please don't name PRs likeFixes #29305. This is not easily understandable. Instead, use a semantic title like "Fix dead links in dev workflow docs". You should then add "Closes#29305" to the description. See also ourPull request guide. - I've adjusted this PR to conform with these rules.

Khushikela29 reacted with thumbs up emoji

@Khushikela29
Copy link
ContributorAuthor

I've tried to resolve the issue, but I guess the checks still didn't pass. I am open to all your suggestions, please guide me where I am doing things wrong since I am in my learning process :)

@rcomer
Copy link
Member

To fix the PR cleanliness check, you can squash the commits into one with aninteractive rebase.

@Khushikela29
Copy link
ContributorAuthor

Thankyou@rcomer for the help :)

@rcomer
Copy link
Member

I think you can also use interactive rebase to drop the "attach_note" commit.

For future PRs, you shouldcreate a separate feature branch, and then you don't get this problem.

@Khushikela29
Copy link
ContributorAuthor

Sure@rcomer I'll keep that in mind.
I am interested in contributing more to the project, it seems fascinating.. if you can help or guide me with more issues in any possible way I'd be grateful.

Copy link
Member

@story645story645 left a comment

Choose a reason for hiding this comment

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

Don't accept my review w/o first dropping the unrelated commit, but the articlethe is missing

@@ -138,13 +138,10 @@ The editing workflow
Note the ``-am`` options to ``commit``. The ``m`` flag signals that you are
going to type a message on the command line. The ``a`` flag stages every
file that has been modified, except files listed in ``.gitignore``. For more
information, see `why the -a flag?`_ and the
`git commit <https://git-scm.com/docs/git-commit>`_ manual page.
information, see `git commit <https://git-scm.com/docs/git-commit>`_ manual page.
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
information, see `git commit<https://git-scm.com/docs/git-commit>`_ manual page.
information, seethe`git commit<https://git-scm.com/docs/git-commit>`_ manual page.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

okay I'll fix it

- Removed unnecessary sentence in the development workflow.- Fixed broken link in `development_workflow.rst`.
@timhoffmtimhoffm merged commit2f51249 intomatplotlib:mainJan 13, 2025
22 checks passed
@QuLogicQuLogic added this to thev3.11.0 milestoneJan 13, 2025
@QuLogicQuLogic mentioned this pull requestJan 15, 2025
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@story645story645story645 left review comments

@timhoffmtimhoffmtimhoffm approved these changes

Assignees
No one assigned
Labels
Documentation: devdocsfiles in doc/develDocumentation: examplesfiles in galleries/examples
Projects
None yet
Milestone
v3.11.0
Development

Successfully merging this pull request may close these issues.

[Doc]: Dead link in dev workflow docs
5 participants
@Khushikela29@timhoffm@rcomer@story645@QuLogic

[8]ページ先頭

©2009-2025 Movatter.jp