-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix PR comments #1460
Fix PR comments #1460
Conversation
|
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
📦 Next.js Bundle Analysis for @faustwp/getting-started-exampleThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Coverage report
Test suite run success121 tests passing in 22 suites. Report generated by 🧪jest coverage report action from 251576b |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Check out the recent updates to your Atlas preview environment:
Learn more about preview environments in our documentation. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🟢 | Statements | 83.56% | 849/1016 |
🟡 | Branches | 68.53% | 516/753 |
🟢 | Functions | 84.4% | 119/141 |
🟢 | Lines | 83.21% | 818/983 |
Test suite run success
121 tests passing in 22 suites.
Report generated by 🧪jest coverage report action from ecbd087
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🟢 | Statements | 84.71% | 72/85 |
🔴 | Branches | 57.14% | 8/14 |
🟡 | Functions | 63.64% | 14/22 |
🟢 | Lines | 83.58% | 56/67 |
Test suite run success
15 tests passing in 5 suites.
Report generated by 🧪jest coverage report action from ecbd087
Tasks
Description
This PR fixes duplicate comments from the next.js bundle analysis action.
You can see from this image the action no longer creates a comment if one already exists
I also opened an issue in the action repo for other users running into this:
hashicorp/nextjs-bundle-analysis#68
Related Issue(s):
Testing
Screenshots
Documentation Changes
Dependant PRs