Skip to content
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

chore: update poetry.lock by running poetry lock -C api (poetry v2) #12737

Closed
wants to merge 2 commits into from

Conversation

chtnnh
Copy link

@chtnnh chtnnh commented Jan 14, 2025

Summary

Fixes #12629
(Unsure if this is a requirement, but the Poetry version used to run poetry lock was 2.0.0)

Screenshots

N/A

Checklist

Important

Please review the checklist below before submitting your pull request.

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. dependencies Pull requests that update a dependency file labels Jan 14, 2025
@laipz8200
Copy link
Member

It seems like there might be some errors. Could you check and regenerate the lock file, please?

@chtnnh
Copy link
Author

chtnnh commented Jan 16, 2025

@laipz8200 sorry, i'm not sure why the tests are failing, anything else I should try?

@laipz8200
Copy link
Member

It seems that the hash in the lock file is not correct. You can try to sync the code with the main branch and run poetry lock --no-update again.

@crazywoola crazywoola mentioned this pull request Jan 22, 2025
8 tasks
@laipz8200
Copy link
Member

blocked by #12369, will check after #12369 merged.

@bowenliang123
Copy link
Contributor

#12369 covers all the necessary changes for poetry v2, including the poetry lockfile.

@laipz8200
Copy link
Member

The changes included in #12369, will close this PR, thank you all.

@laipz8200 laipz8200 closed this Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

poetry.lock requires regeneration when install locally from source code
3 participants