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

Support pull_request_target #31

Closed
wants to merge 1 commit into from
Closed

Conversation

Ana06
Copy link

@Ana06 Ana06 commented May 19, 2021

pull_request_target is granted a read/write repository token and runs in the context of the base of the pull request. It allows to create workflows that label and comment on pull requests based on the contents of the event payload. The get-changed-files action is useful for some of those actions and the code works for this case with almost no modification.

You can see it working here:

You can try it using:

- name: Get changed files
  id: files
  uses: Ana06/get-changed-files@master

Example: Ana06/capa@ede0ddf

Supersedes #16 and #13 (I think dist needed to be updated in the first PR and it is not needed to delete the "head commit is ahead of the base commit" check in the other).

A release with this change would be appreciated. 😄

`pull_request_target` is granted a read/write repository token and runs
in the context of the base of the pull request. It allows to create
workflows that label and comment on pull requests based on the contents
of the event payload. The `get-changed-files` action is useful for some
of those actions and the code works for this case with almost no
modification.
@Ana06 Ana06 closed this May 26, 2021
@Ana06 Ana06 deleted the branch jitterbit:master May 26, 2021 14:35
@Ana06 Ana06 deleted the master branch May 26, 2021 14:35
@Ana06
Copy link
Author

Ana06 commented May 26, 2021

It seems this project is not maintained anymore or there is no interest on adding this. I have released Ana06/get-changed-files@v1.1 which includes this change, #27 and fixes the tests. Feel free to cherry-pick the commits if you want.

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.

1 participant