codecov / codecov/codecov-javascript-bundler-plugins
significant delay in GitHub PR comment
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 9
- Forks
- 10
- PR merge metrics
- No merged PRs in 30d
Description
📣 Feedback / 🐛 Bugs
Do you want to file a bug report and/or feature request for Codecov? Please use our feedback repo instead.
When using the javascript bundle analysis tool, there is a significant delay in posting the PR comment on GitHub.
Here are several examples where the PR comment was posted hours after I merged the actual PR. All the checks were completed when I merged, except the bundle analysis comment.
- https://github.com/LizardByte/Sunshine/pull/4510
- https://github.com/LizardByte/Sunshine/pull/4595
- https://github.com/LizardByte/Sunshine/pull/4594
- https://github.com/LizardByte/Sunshine/pull/4592
TIP: when viewing the comments, you have hover over the relative time, to see the actual time.
I also use the codecov GitHub action for coverage and test results, and it should be noted that there are no noticeable delays when the comment from that action is posted.
Contributor guide
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
Review the javascript bundle analysis plugin's GitHub PR comment flow and the four linked pull requests, comparing their merge, check-completion, and comment timestamps. Start by tracing where the bundle analysis result is sent for comment publication; done means identifying and correcting the source of the hours-long delay, with timely comments confirmed against similar pull requests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, typescript
- Domain
- ci-cd
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100