cypress-image-snapshot icon indicating copy to clipboard operation
cypress-image-snapshot copied to clipboard

Improvement/update node support to 18 and cypress 10 11 12

Open sosog opened this issue 2 years ago • 18 comments
trafficstars

I have upgraded a few packages to support cypress 10 and also 11 and 12 versions, with nodejs 16.x and 18.x. Babel also should be updated, maybe deep refactoring of js files to ts and removing some packages in the future would be good decisions, otherwise working with this package going to be a headache. I have updated jest-image-snapshot to 5.0.0. version and example react project too, with jest-image-snapshot i had multiple issues which need a lot of time I think to fix so I don't know if there is any sense to do such big job if cypress-image-snapshot packages going to die instead of updating.

sosog avatar Dec 09 '22 12:12 sosog

🦋 Changeset detected

Latest commit: 095d547a322cc39bb20e0b2f206d6e181a3bbe9a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
cypress-image-snapshot Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Dec 09 '22 12:12 changeset-bot[bot]

@jaredpalmer may I ask who is maintaining this package? I have some small changes id like to make + pr's like this need some love I think! Any chance its worth adding more contributors to give this project some life? Cheers

paperschool avatar Dec 28 '22 13:12 paperschool

@paperschool as you can see, no one answered, I have published my fork into NPM, but better to create a new package from scratch. I also found other PR so if there will be interest from other devs we can try to write a package with a modern environment and starter kits.

sosog avatar Dec 30 '22 16:12 sosog

@sosog would you say the contents of this package are sufficiently out of date as to not make it worth keeping and republishing? Id be game on recreating it but frankly I am not sure on the intricacies of the entire package. What is worth keeping and what isnt etc.

paperschool avatar Jan 01 '23 16:01 paperschool

@paperschool I say it as an option only, because authors are inactive, as you can see I have updated it already, in the future we can switch to TS and just move the logic of this package, but anyway, it's tons of work.

sosog avatar Jan 03 '23 06:01 sosog

@jaredpalmer could you help us with this PR?

everywebco avatar Jan 12 '23 09:01 everywebco

@jackjocross @MichaelDeBoey @lazarljubenovic

sosog avatar Jan 12 '23 11:01 sosog

@simonsmith you have similar PR

sosog avatar Jan 12 '23 11:01 sosog

I like the sound of a new package and we can drop all the forks

simonsmith avatar Jan 13 '23 13:01 simonsmith

@jackjocross @MichaelDeBoey @lazarljubenovic

What?

lazarljubenovic avatar Jan 16 '23 06:01 lazarljubenovic

@jackjocross @MichaelDeBoey @lazarljubenovic

What?

You are contributor, maybe you have contact with the package author, he is inactive here.

sosog avatar Jan 16 '23 07:01 sosog

I have tried contacting Jared directly on Twitter but no joy. Safe to assume this package will not receive any more updates

simonsmith avatar Jan 16 '23 10:01 simonsmith

@lazarljubenovic not funny...

sosog avatar Jan 22 '23 13:01 sosog

@simonsmith so we can create maybe some discord chat, to talk about perspectives

sosog avatar Jan 22 '23 13:01 sosog

Just fork the goddamn package.

lazarljubenovic avatar Jan 22 '23 13:01 lazarljubenovic

Just fork the goddamn package.

I don't know where you grew up, but you should be more polite here, and yes this one is already forked, that's the main topic. Bye.

sosog avatar Jan 25 '23 18:01 sosog

I don't know where you grew up, but having a message consist solely of three pings is considered spam. Posting three messages in a row as well.

lazarljubenovic avatar Jan 25 '23 22:01 lazarljubenovic

@simonsmith deprecated in prefer of your package https://www.npmjs.com/package/@sosog/cypress-image-snapshot

sosog avatar Jul 31 '23 18:07 sosog