-
-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
chore: setup Argos Visual Testing #5577
chore: setup Argos Visual Testing #5577
Conversation
✅ Deploy Preview for mermaid-js 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 @@
## develop #5577 +/- ##
==========================================
- Coverage 5.73% 5.73% -0.01%
==========================================
Files 277 278 +1
Lines 41988 41999 +11
Branches 515 490 -25
==========================================
Hits 2409 2409
- Misses 39579 39590 +11
Flags with carried forward coverage won't be shown. Click here to find out more. |
The latest updates on your projects. Learn more about Argos notifications ↗︎ Waiting for the first build to start… |
b6f4bd1
to
d705a16
Compare
@sidharthv96 @knsv could you give a look? |
@sidharthv96 how can I help? |
0c78232
to
493bb8a
Compare
* develop: Removed unused patch Fix cytoscape patch chore: Remove cytoscape patch. docs: fix node version in CONTRIBUTING.md Fixed entrypoint path
e951560
to
1f70717
Compare
@gregberge, Thank you for the contribution! |
📑 Summary
As discussed with @sidharthv96, I setup Argos Visual Testing. At some point you should be able to replace Applitools without any limitation of usage.
📏 Design Decisions
I kept Applitools and
cy.matchImageSnapshot
, you should be able to remove it once Argos fits your need!📋 Tasks
Make sure you
MERMAID_RELEASE_VERSION
is used for all new features.develop
branch