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

Roll python deps & add pigweed's #4168

Merged
merged 1 commit into from
Dec 15, 2020

Conversation

mspang
Copy link
Contributor

@mspang mspang commented Dec 11, 2020

This will help avoid issues like the one fixed by #4165, where a pip
package used by pigweed updated underneath us and broke.

These are gathered from install_requires in setup.py of packages we
install. Currently we don't have a way to fail the build if there's an
unpinned dep; they'll have to be found manually and added to
requirements.in.

@mspang mspang force-pushed the for-chip/roll-python-deps branch from 585a159 to fb2bdcb Compare December 11, 2020 01:51
This will help avoid issues like the one fixed by project-chip#4165, where a pip
package used by pigweed updated underneath us and broke.

These are gathered from install_requires in setup.py of packages we
install. Currently we don't have a way to fail the build if there's an
unpinned dep; they'll have to be found manually and added to
requirements.in.
@mspang
Copy link
Contributor Author

mspang commented Dec 15, 2020

@mspang mspang merged commit 4e78f50 into project-chip:master Dec 15, 2020
@mspang mspang deleted the for-chip/roll-python-deps branch December 15, 2020 20:44
kpschoedel pushed a commit to kpschoedel/connectedhomeip that referenced this pull request Jan 8, 2021
This will help avoid issues like the one fixed by project-chip#4165, where a pip
package used by pigweed updated underneath us and broke.

These are gathered from install_requires in setup.py of packages we
install. Currently we don't have a way to fail the build if there's an
unpinned dep; they'll have to be found manually and added to
requirements.in.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants