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

chore: only run check pr action on pull_request_target events #485

Merged
merged 1 commit into from
Sep 10, 2023

Conversation

kdheepak
Copy link
Collaborator

@kdheepak kdheepak commented Sep 10, 2023

From the documentation: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target

This event runs in the context of the base of the pull request, rather than in the context of the merge commit, as the pull_request event does. This prevents execution of unsafe code from the head of the pull request that could alter your repository or steal any secrets you use in your workflow. This event allows your workflow to do things like label or comment on pull requests from forks. Avoid using this event if you need to build or run code from the pull request.

@codecov
Copy link

codecov bot commented Sep 10, 2023

Codecov Report

Merging #485 (69b2302) into main (322e46f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #485   +/-   ##
=======================================
  Coverage   90.16%   90.16%           
=======================================
  Files          41       41           
  Lines       11277    11277           
=======================================
  Hits        10168    10168           
  Misses       1109     1109           

@kdheepak kdheepak merged commit af36282 into main Sep 10, 2023
36 checks passed
@kdheepak kdheepak deleted the pull_request_target branch September 10, 2023 09:19
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