Open xerxesb opened 3 years ago
Hi, @xerxesb, Thanks for your contribution :blue_heart: . Contributors :people_holding_hands: like you make the open source community :earth_africa: such an amazing place to learn :book: , inspire :angel:, and create :art: . We will investigate :eyes: and get back to you as soon as possible :+1: . Just make sure you have given us sufficient information :information_source:.
By that time enjoy this meme :point_down: , hope you like it :smile:
Use this action on your projects. Use jokes on issues instead.
unfortunately, this action does not work on PRs, as it does not have the write permission to write the report back on the pull request's codebase. I'm working on a bot that overcomes this. stay tuned for it.
Thanks. Appreciate if you could drop a comment in this issue when it's available!
yeah sure :)
The check is still fundamentally wrong though because (especially with organization repositories), the source-branch might be in the same repo as the destination-branch.
On top of that, the action can still be useful without writing a report. just failing the status-check to prevent merging when branch-protection is enabled is good enough.
Also, it's not the actions task to do such a check, the user of the action is the only person who can know if the provided token works on the repo or not and should enable or disable the action using if:
.
IMO you should just remove the check without changing anything else.
What's the status of this issue? It's been some time since the last update and I was wondering if some work has been done in the meanwhile.
I forked over at https://github.com/chmorgan/cppcheck-action and removed this check. Any help would be appreciated!
I'm not a GHA expert but rather than committing the report. You could:
I have added this cppcheck action to one of my workflows, and I've noticed that the job executes on push, but it doesn't execute on pull request. In both cases its the exact same job - one runs, the other doesn't.
My job definition:
When the job executes as part of a
push
, I get the full output and everything works.When the job executes as part of a
pull_request
I only get the following in the log:I would imagine this is a bug? There shouldn't be a reason why it runs on commit but not on PR?