Skip to content

Conversation

@hlky
Copy link
Contributor

@hlky hlky commented Mar 11, 2025

What does this PR do?

Adds make fix-copies to style bot.

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@hlky hlky requested a review from sayakpaul March 11, 2025 11:35
echo "No changes in the files. Proceeding..."
rm -rf main_Makefile main_setup.py main_check_doc_toc.py
style_command: "make style && make quality"
style_command: "make style && make quality && make fix-copies"
Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/huggingface/diffusers/blob/4e3ddd5afab3a4b0b6265f210d6710933dade660/Makefile#L70C2-L71C51

We should also make sure the PR doesn't have any changes in the Python files we use in fix-copies. However, there's some discussions in https://github.com/huggingface/huggingface_hub/pull/2914/files#r1987369608. So, maybe let's wait a bit before we merge this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, yes let's wait until those discussions are resolved.

@hlky hlky marked this pull request as draft March 11, 2025 15:26
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions bot added the stale Issues that haven't received updates label Apr 10, 2025
@hlky hlky closed this Apr 15, 2025
@hlky hlky deleted the bot-fix-copies branch April 15, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Issues that haven't received updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants