Skip to content

Conversation

@legobeat
Copy link
Contributor

@legobeat legobeat commented May 18, 2023

The latest publicly released version of madge has a runtime dependency of typescript@^3.9.5. #350 moved it to peerDependencies and lifted the version range to *.

This PR proposes to constrain the typescriptversions supported in peerDependencies to either of:

  • ^3.9.5 (backwards-compat with madge@6.0.0)
  • ^5 (supporting current version makes sense if it already works)
  • ^4.9.5 (only makes sense considering the above but don't support legacy 4.x)

The intention here is to reduce the range of supported versions to make it more feasible to test for, as well as the risk of future regressions or unplanned breaking changes.

@legobeat legobeat marked this pull request as ready for review May 18, 2023 20:51
@legobeat legobeat mentioned this pull request May 18, 2023
@legobeat
Copy link
Contributor Author

@PabloLION @kamiazya Got a rough idea of if/when this can be looked at?

Copy link
Collaborator

@kamiazya kamiazya left a comment

Choose a reason for hiding this comment

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

Sorry for the slow response, I've been busy.

LGTM!

@kamiazya kamiazya merged commit c48c516 into pahen:master Jun 1, 2023
@kamiazya kamiazya added this to the [Planning]6.1.0 milestone Jun 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants