Skip to content

bazel: Bump rules_python to 0.3.0#17976

Merged
mattklein123 merged 2 commits intoenvoyproxy:mainfrom
phlax:bazel-rules-python-bump
Sep 3, 2021
Merged

bazel: Bump rules_python to 0.3.0#17976
mattklein123 merged 2 commits intoenvoyproxy:mainfrom
phlax:bazel-rules-python-bump

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 3, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: bazel: Bump rules_python to 0.3.0
Additional Description:

This brings in a lot of updates, particularly pip_install_incremental - which iiuc will allow us to cut down duplication of deps and speed up ci

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Ryan Northey <ryan@synca.io>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 3, 2021
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #17976 was opened by phlax.

see: more, trace.

Bumps [pytest](https://github.com/pytest-dev/pytest) from 6.2.4 to 6.2.5.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@6.2.4...6.2.5)

---
updated-dependencies:
- dependency-name: pytest
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 3, 2021

this pr requires a pip dep update, as afaict there is a dependency conflict with upstream packages in current requirements

@phlax phlax requested a review from moderation September 3, 2021 08:22
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 3, 2021

...as such it may conflict with #17977

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Sep 3, 2021
@mattklein123 mattklein123 merged commit 8b280c7 into envoyproxy:main Sep 3, 2021
tyxia pushed a commit to tyxia/envoy that referenced this pull request Sep 21, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>

Signed-off-by: phlax <phlax@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants