Skip to content

Conversation

Sai-Suraj-27
Copy link
Contributor

What does this PR do?

Fixes a wrong link to supported Python versions in contributing.md file.

Before submitting

Who can review?

Core library:

CONTRIBUTING.md Outdated
@@ -355,7 +355,7 @@ You will need basic `git` proficiency to be able to contribute to
manual. Type `git --help` in a shell and enjoy. If you prefer books, [Pro
Git](https://git-scm.com/book/en/v2) is a very good reference.

Follow these steps to start contributing ([supported Python versions](https://github.com/huggingface/diffusers/blob/main/setup.py#L265)):
Follow these steps to start contributing ([supported Python versions](https://github.com/huggingface/diffusers/blob/main/setup.py#L270)):
Copy link
Member

Choose a reason for hiding this comment

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

Actually, this should ideally be a permalink.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, it makes more sense. I updated the link to a permalink. Please check now. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

It will be only needed to update this permalink when supported Python versions are changed. Make sense 👍.

Copy link
Contributor

@tolgacangoz tolgacangoz left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!
Actually, it was me to make it a temporary link last time 😅.

@Sai-Suraj-27 Sai-Suraj-27 requested a review from sayakpaul May 12, 2024 11:11
@sayakpaul sayakpaul merged commit ec9e881 into huggingface:main May 12, 2024
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
….md` file (#7638)

* Fixed a wrong link to python versions in contributing.md file.

* Updated the link to a permalink, so that it will permanently point to the specific line.
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.

3 participants