fktn-k / fkYAML

A C++ header-only YAML library
MIT License
67 stars 7 forks source link

Fix workflow errors on PRs from fork repos #383

Closed fktn-k closed 1 month ago

fktn-k commented 1 month ago

With this PR, I try to fix workflow errors on a PR from a fork repository since some workflows need to write permission to the PR or acceess to secrets to push an automatically created commit.
One way to resolve them is change the trigger event from pull_request to pull_request_target, just as this PR does.
Note, according to the official document, that can be insecure against stealing secrets or running a malicious code.
So, further changes will be required to increase security in the future since this is first aid to the errors.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't