-
Notifications
You must be signed in to change notification settings - Fork 119
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
Maintaining this plugin #210
Comments
Absolutely agree with @GabrielMcNeilly. This plugin causing crashes on my cypress because its specific dependency to cypress 4.5.0. It should have been sth like cypress > 4.5.0 maybe? |
I am also curious about the future of the plugin. I can help out - I have deep knowledge on Cypress internals. I'd say the first step would be updating to each major Cypress version. The best option might simply be to fork or move the plugin to it's own org, if the original author is no longer using it or able to maintain it. |
See this thread: #194 I think there is still an issue with the github actions with are blocking PRs. I had a look at it last year but got a bit stuck. Maybe you have an idea? I think @joetidee and @edimitchel have permission to merge PRs. |
Not anymore, PR are failing and we cannot override PR restriction in case of PR build failing.. |
What does @meinaart say about that? Are you still interested in maintaining the plugin? Otherwise it might be a good idea to transfer the ownership to someone else. @lmiller1990 seems to be the perfect fit 😅 @dhulme What is the issue with the github actions? |
We've now migrated to https://github.com/FRSOURCE/cypress-plugin-visual-regression-diff which works fine with Cypress 10 🥳 |
@dhulme How difficult was the migration to https://github.com/FRSOURCE/cypress-plugin-visual-regression-diff? I'm trying to set up github actions for my org's cypress testing and it looks like we're going to need an alternative to cypress-plugin-snapshots. My organization is still using v8.4.1, if that matters. |
It's pretty simple. It uses the same folder structure for snapshots. I'd suggest updating to cypress 10 first. You may need to regenerate all the screen shots as they were slightly different dimensions for me.
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: david-renner-dwg ***@***.***>
Sent: Tuesday, August 16, 2022 10:26:53 PM
To: meinaart/cypress-plugin-snapshots ***@***.***>
Cc: Dave Hulme ***@***.***>; Mention ***@***.***>
Subject: Re: [meinaart/cypress-plugin-snapshots] Maintaining this plugin (Issue #210)
@dhulme<https://github.com/dhulme> How difficult was the migration to https://github.com/FRSOURCE/cypress-plugin-visual-regression-diff? I'm trying to set up github actions for my org's cypress testing and it looks like we're going to need an alternative to cypress-plugin-snapshots. My organization is still using v8.4.1, if that matters.
—
Reply to this email directly, view it on GitHub<#210 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAJNN32RWPD3J7RFDGYUGT3VZQBR3ANCNFSM5QM7TPAQ>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@meinaart Can you give me/someone/others permissions to approve/merge PRs etc for this project?
It's a pretty nice plugin but it is in need of a fair amount of updates. I and others would be interested in contributing/maintaining it if you are no longer able to do so.
Currently it seems it is being left to stagnate and will soon be unusable. Given there is no fantastic visual testing Cypress plugin imho, this has the opportunity to be the main one so it seems a waste to just leave it especially when there are people willing to contribute.
The text was updated successfully, but these errors were encountered: