Skip to content

Reflect few contributions on contribution.md that were not reflected on #8294 #8938

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

Merged
merged 22 commits into from
Aug 19, 2024

Conversation

mreraser
Copy link
Contributor

@mreraser mreraser commented Jul 23, 2024

What does this PR do?

Hello!😃

As mentioned in the previous PR, this PR includes the incorporation of unaddressed contributions to evaluation.md and contribution.md. Upon review, I found that all contributions were already applied to evaluation.md. For the contribution.md file, I made corrections to the numbering and added a TOC as instructed in the last PR.

Before submitting

Who can review?

@heuristicwave

Thank you always for reviewing! 🤗

Best regards,
Jiwook Han

mreraser and others added 3 commits July 26, 2024 17:01
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Copy link
Contributor

@heuristicwave heuristicwave left a comment

Choose a reason for hiding this comment

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

The document was very long, so the review took a long time. Thanks to this, I was able to read the contribute document again and learn what I should take to heart.

@mreraser
Copy link
Contributor Author

The document was very long, so the review took a long time. Thanks to this, I was able to read the contribute document again and learn what I should take to heart.

Thank you so much for your continuous and dedicated reviews. Your feedback has been invaluable, and I truly appreciate the time and effort you've invested in.

Have a nice day😊

mreraser and others added 5 commits August 17, 2024 15:06
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
mreraser and others added 4 commits August 17, 2024 15:06
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
@mreraser mreraser marked this pull request as ready for review August 17, 2024 07:34
@mreraser
Copy link
Contributor Author

Hello @stevhliu 😊, may I ask for your final review of the latest changes when you have a moment? If everything looks good, I would greatly appreciate it if you could merge them. Thank you so much! 🙏

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@stevhliu stevhliu 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 following up and completing! 🤗

@stevhliu stevhliu merged commit d72bbc6 into huggingface:main Aug 19, 2024
1 check passed
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
…d on #8294 (#8938)

* incorrect_number_fix

* add_TOC

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* fix: manual edits

* fix: manual edtis

* fix: manual edits

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* Update docs/source/ko/conceptual/contribution.md

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>

* fix: manual edits

---------

Co-authored-by: Jihun Lim <31366038+heuristicwave@users.noreply.github.com>
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.

4 participants