lincc-frameworks / lincc-frameworks/asv-formatter
Fail gracefully for forked PRs
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 2
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
If we can't update the comments, we should just continue to dump out the ASV results from the workflows without failing to update the comment.
This is not related to this repo, per se, but we should find a solution for github workflows on PRs that merge from a fork.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the GitHub workflows that process and publish ASV results, focusing on the comment-update step for pull requests from forks. Determine how a comment-update failure currently affects workflow output. Done means forked PR workflows still dump the ASV results and do not fail solely because the comment cannot be updated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, python
- Domain
- ci-cd
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100