Skip to content

tools: Restrict check-deps workflow to run only on the main repo#18644

Merged
RyanTheOptimist merged 1 commit intoenvoyproxy:mainfrom
RyanTheOptimist:check-deps
Oct 15, 2021
Merged

tools: Restrict check-deps workflow to run only on the main repo#18644
RyanTheOptimist merged 1 commit intoenvoyproxy:mainfrom
RyanTheOptimist:check-deps

Conversation

@RyanTheOptimist
Copy link
Copy Markdown
Contributor

Signed-off-by: Ryan Hamilton rch@google.com

tools: Restrict check-deps workflow to run only on the main repo

Commit Message: N/A
Additional Description: N/A
Risk Level: N/A
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Ryan Hamilton <rch@google.com>
@RyanTheOptimist
Copy link
Copy Markdown
Contributor Author

/assign @phlax

@RyanTheOptimist
Copy link
Copy Markdown
Contributor Author

@phlax I'm seeing emails about the check-deps workflow failing to run in my fork. I think we need it to run only in the main repo just like we did with the PR notifier in #18592. WDYT?

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks @RyanTheOptimist

if only we could get dependabot to respect this too

@RyanTheOptimist RyanTheOptimist merged commit 82120a2 into envoyproxy:main Oct 15, 2021
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.

2 participants