Skip to content

Conversation

@MichaelDeBoey
Copy link
Member

CI is failing because ts-jest changed their peerDependencies (which should have been a major release)

@MichaelDeBoey MichaelDeBoey added the dependencies Pull requests that update a dependency file label Aug 21, 2021
@MichaelDeBoey MichaelDeBoey requested a review from Belco90 August 21, 2021 17:16
@MichaelDeBoey MichaelDeBoey enabled auto-merge (squash) August 21, 2021 17:19
Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

3rd time this year we get CI broken because of new package version released introduced unexpected breaking changes.

Thinking seriously now about fixing versions in package.json

@MichaelDeBoey MichaelDeBoey merged commit e1f39fc into main Aug 22, 2021
@MichaelDeBoey MichaelDeBoey deleted the update-dependencies branch August 22, 2021 14:16
@MichaelDeBoey
Copy link
Member Author

MichaelDeBoey commented Aug 22, 2021

I don't think we should fix versions in our package.json as CI failing is happening really rarely.
ts-jest should have made kulshekhar/ts-jest#2816 a breaking change.

Fixing versions will break the dedupe feature too.

Edit: it appears ts-jest doesn't apply SemVer, so we should fix that version I think
https://github.com/kulshekhar/ts-jest#versioning

@github-actions
Copy link

🎉 This PR is included in version 4.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants