-
Notifications
You must be signed in to change notification settings - Fork 544
chore: alphabetically sort dev-requirements.txt #4890 #4893
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments below. Looks like you had an unresolved diff, probably because of an end of file whitespace difference.
Apologies for the confusion earlier! I mistakenly caused some merge conflicts while sorting the file alphabetically.
Thank you for your patience, and I appreciate the feedback! 🤗 |
I have kicked off CI, will be back once they run. |
@AshishYesale7 It seems you got merge conflict from this PR you need to fix them and update the branch. |
Ugh, sorry about the merge conflict; I'd forgotten that our monthly pre-commit update would cause problems for you. Let me know if you need help resolving it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove the sync-upstream.yml file from this PR? It's not related to this PR and it seems like it's a personal branch thing for you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we're all ready to merge! Thank you @AshishYesale7 and congratulations on your first merged commit with us!
This PR sorts the
dev-requirements.txt
file alphabetically to improve readability.Fixes #4890