Skip to content

tools: setup GHA for PR notifier#16756

Closed
mattklein123 wants to merge 1 commit intomainfrom
pr_gha
Closed

tools: setup GHA for PR notifier#16756
mattklein123 wants to merge 1 commit intomainfrom
pr_gha

Conversation

@mattklein123
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Matt Klein <mklein@lyft.com>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Jun 1, 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: #16756 was opened by mattklein123.

see: more, trace.

@mattklein123
Copy link
Copy Markdown
Member Author

@alyssawilk I think it's pretty much as easy as this. Right now I have it running on PR just to test it, but then we can switch it over to both a cron and manual dispatch. I can also help you with the secrets offline.

@mattklein123
Copy link
Copy Markdown
Member Author

cc @phlax also

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r ./.github/actions/pr_notifier/requirements.txt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1 - the pip installer i linked doesnt support require-hashes afaict - this sees to be the most straightforward

@alyssawilk
Copy link
Copy Markdown
Contributor

Nice!!

@daixiang0
Copy link
Copy Markdown
Member

I am not sure whether it can be replaced by this tool.

@mattklein123
Copy link
Copy Markdown
Member Author

I'm just going to close this. @alyssawilk this is simple enough let's just merge this into your other PR?

@mattklein123 mattklein123 deleted the pr_gha branch June 2, 2021 02:11
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.

4 participants