Skip to content

Conversation

@DamnedScholar
Copy link

This should fix #147.

I had to update package.json to pin @types/jest to ^27.0.0 to resolve a dependency conflict on install. Every other change is directly related to the issue fix.

Copy link
Collaborator

@schemar schemar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove package-lock.json and use yarn instead. Yarn should also use the existing yarn.lock file where dependencies should have their correct versions.

@schemar
Copy link
Collaborator

schemar commented Jan 2, 2022

Thank you @DamnedScholar, looks very compact/nice 👍 I don't have much time right now, but I will have a look as soon as I do.

Copy link
Collaborator

@schemar schemar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I didn’t elaborate on that enough with my last comment. I think if you revert package.json and yarn.lock, then do a yarn install, everything should work. Or is that not the case?

@schemar schemar added the question Further information is requested label Feb 16, 2022
@schemar
Copy link
Collaborator

schemar commented Mar 27, 2022

@DamnedScholar you still have unrelated changes in your package.json and yarn.lock.

@claremacrae
Copy link
Collaborator

Setting to draft status any PRs which are not currently mergable, but will be useful in future.

@claremacrae claremacrae marked this pull request as draft May 7, 2024 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

3 participants