-
Notifications
You must be signed in to change notification settings - Fork 447
[refactor] Do not use pinned versions in requirements.txt #4714
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
7e361b3 to
39d6161
Compare
|
@bruntib Can you please support? UPDATE: The failing build also fails on the Unfortunately, some of codechecker's python dependencies are quite outdated. As a result, users installing with Would you maybe have some time to update and integrate the newer pip dependencies when possible? |
f24181a to
1b0c655
Compare
|
I checked that the issue is resolved as follows (using nodejs v25.2.0): I can verify that this works and e.g. setuptools is in newer version |
It is not recommended by the Python Packaging User Guide to pin dependencies. Instead, use minimum or maximum versions wherever possible. Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be> Signed-off-by: Thorsten Klein <thorsten.klein@bshg.com>
analyzer/tools/merge_clang_extdef_mappings/requirements_py/dev/requirements.txt analyzer/tools/statistics_collector/requirements_py/dev/requirements.txt
analyzer/tools/merge_clang_extdef_mappings/requirements_py/dev/requirements.txt analyzer/tools/statistics_collector/requirements_py/dev/requirements.txt analyzer/tools/statistics_collector/requirements_py/dev/requirements.txt
1b0c655 to
101a068
Compare
|
If anybody else is stumbling across this: We have workarounded this issue by switching to The |
|
Hi @thorsten-klein, Sorry for being this slow with this PR. We didn't want to change dependency versions in the last release at the end of the year, but it is clear that a better upgrade procedure is neccessary due to these conflict issues. So, thank you for the improvement, I'll merge it. Also, I'm working on upgrading further dependencies, and pinning the major version only. I believe, the |
Follow-up of #4501.
It is not recommended by the Python Packaging User Guide to pin dependencies. Instead, use version ranges or minimum/maximum versions wherever possible.
Why?
With zephyr 4.3.0 I get some conflict: