Skip to content

Conversation

@jackfirth
Copy link
Contributor

See the comment in resyntax-analyze.yml for why.

See the comment in `resyntax-analyze.yml` for why.
@samth
Copy link
Member

samth commented Jul 9, 2022

Don't you need to use pull_request_target somewhere?

@jackfirth jackfirth merged commit f83137d into racket:master Jul 9, 2022
@jackfirth
Copy link
Contributor Author

...oops. Yes I do. Will fix that in a sec.

@jackfirth
Copy link
Contributor Author

@samth wait no, actually I don't need to use pull_request_target anymore. Instead, I use workflow_run. The second workflow runs when the first one finishes, and the first one doesn't need pull_request_target because it no longer needs write access to the repository.

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