Despite having an optional HeadSHA field, if you update this to a different commit hash it won't be updated.
We have some Check Runs that are PR-oriented and not commit-oriented and we were trying to use this field to rebind the Check Runs to the head commit of a PR on synchronize.
Is this a bug?
As far as I can tell from the official documentation: https://docs.github.com/en/free-pro-team@latest/rest/reference/checks#update-a-check-run
the head_sha field is no longer accepted as a parameter and is completely ignored.
(Therefore, we probably ought to remove this field from the struct entirely.)
You may wish to contact GitHub technical support and ask them the same question.
+1 to removing HeadSHA from the UpdateCheckRunOptions struct to prevent further confussion.
The other fields should be verified as well as part of addressing this issue.
This would be a great PR for any new contributor to this repo or a new Go developer.
All contributions are greatly appreciated!
Feel free to volunteer for any issue and the issue can be assigned to you so that others don't attempt to duplicate the work.
Please check out our CONTRIBUTING.md guide to get started.
Thank you!
Hi @gmlewis @Adirio I can work on this issue. Can anyone of you please assign it to me?
Please take a look at PR-1657 . It will be my first contribution to this project :)
Most helpful comment
Please take a look at PR-1657 . It will be my first contribution to this project :)