Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reflect few contributions on philosophy.md that were not reflected on #8294 #8690

Merged
merged 5 commits into from
Jul 2, 2024

Conversation

mreraser
Copy link
Contributor

What does this PR do?

Hello!😀

In the previous PR, I mistakenly named the file philosopy.md and later corrected it to philosophy.md.

However, during this process, it was identified by @heuristicwave that the reviews provided by @jungnerd were not reflected in the PR. I apologize for the oversight and any inconvenience caused. I should have double-checked before the PR was merged.

I am resubmitting the PR with the latest changes included. Thank you for your understanding.

from #7940

Before submitting

Who can review?

@heuristicwave @stevhliu

Some contributions were not reflected previously, so I am resubmitting them.
@sayakpaul sayakpaul requested review from yiyixuxu and stevhliu and removed request for yiyixuxu June 25, 2024 07:37
@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 coming back to apply those fixes! Feel free to ping me for another review once all the feedback from @jungnerd has been incorporated 🙂

mreraser and others added 3 commits June 26, 2024 01:52
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
@heuristicwave
Copy link
Contributor

@mreraser Looks good this time, thanks for the fix, change draft to open and request a merge!

@mreraser mreraser marked this pull request as ready for review July 2, 2024 02:52
@mreraser
Copy link
Contributor Author

mreraser commented Jul 2, 2024

@mreraser Looks good this time, thanks for the fix, change draft to open and request a merge!

Thank you @heuristicwave for your review! Have a nice day 👍

@stevhliu stevhliu merged commit dd4b731 into huggingface:main Jul 2, 2024
1 check passed
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
#8294 (#8690)

* Update philosophy.md 

Some contributions were not reflected previously, so I am resubmitting them.

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

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

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

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

---------

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants