-
Notifications
You must be signed in to change notification settings - Fork 142
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
build: remove vrt
from ci.yml and run directly from netlify build script
#6578
build: remove vrt
from ci.yml and run directly from netlify build script
#6578
Conversation
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ibm-products-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6578 +/- ##
==========================================
- Coverage 79.96% 79.95% -0.02%
==========================================
Files 394 394
Lines 12899 12899
Branches 4274 4274
==========================================
- Hits 10315 10313 -2
- Misses 2584 2586 +2
|
vrt
from ci.yml and run directly from netlify build script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
20f5787
Run percy from netlify build script to avoid using hardcoded percy token directly in
.github/workflows/ci.yml
.What did you change?
How did you test and verify your work?
Verified snapshots were still being taken from netlify logs.