Skip to content

Conversation

viktorrusakov
Copy link
Contributor

We've tried using this config in Paragon, but installing the package causes errors with peerDependencies as Paragon already has @types/jest and ts-jest installed with higher versions.

As far as I understand typescript config doesn't really need ts-jest nor @types/jest to run (it certainly doesn't care about their versions), so I think we should just delete them.

@openedx-webhooks openedx-webhooks added the blended PR is managed through 2U's blended developmnt program label May 12, 2023
@openedx-webhooks
Copy link

Thanks for the pull request, @viktorrusakov!

When this pull request is ready, tag your edX technical lead.

@abdullahwaheed abdullahwaheed merged commit 5e08233 into master May 12, 2023
@abdullahwaheed abdullahwaheed deleted the rusakov/remove-redundant-peer-dependencies branch May 12, 2023 13:46
@openedx-webhooks
Copy link

@viktorrusakov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

@edx-semantic-release
Copy link

🎉 This PR is included in version 1.0.1 🎉

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
blended PR is managed through 2U's blended developmnt program released
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants