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

RLQS: Fix CEL dep in bzl #29195

Merged
merged 2 commits into from
Aug 22, 2023
Merged

RLQS: Fix CEL dep in bzl #29195

merged 2 commits into from
Aug 22, 2023

Conversation

tyxia
Copy link
Member

@tyxia tyxia commented Aug 22, 2023

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: tyxia <[email protected]>
@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #29195 was opened by tyxia.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 22, 2023
@repokitteh-read-only
Copy link

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).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #29195 was opened by tyxia.

see: more, trace.

@tyxia tyxia marked this pull request as ready for review August 22, 2023 18:15
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Aug 22, 2023
Signed-off-by: tyxia <[email protected]>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 22, 2023
@RyanTheOptimist RyanTheOptimist enabled auto-merge (squash) August 22, 2023 19:27
@RyanTheOptimist RyanTheOptimist merged commit 9f26b8c into envoyproxy:main Aug 22, 2023
118 of 119 checks passed
@tyxia tyxia deleted the cel_dep_rlqs branch August 22, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps Approval required for changes to Envoy's external dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants