Skip to content

remove typescript peer dependency #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

mcm1957
Copy link
Contributor

@mcm1957 mcm1957 commented Mar 8, 2025

This PR removes peerDependency for typescript

Requiring typescript to be ~5.8.0 blocks installation of eslint-config@2.0.0 as log as an older typescript version is installed. Otherwise updating typescript to 5.8.x is not possible as long as eslint-confi@1.x.x is installed - at least not without ignoring errors.

Until 2.0.0 typescript was not listed as peerDependency. I do not see a reason to request / check this as eslint typeScript plugin(s) anyway check the supported typescript version. And eslint-config itself does not use tyoescript or an special feature of it.

So eslint-config should not require a specific typescript version to allow a smooth installtion.

@foxriver76
Copy link
Contributor

Can you add changelog placeholder?

@mcm1957 mcm1957 mentioned this pull request Mar 9, 2025
@mcm1957 mcm1957 changed the title remove typescript peer dependendy remove typescript peer dependency Mar 9, 2025
@mcm1957
Copy link
Contributor Author

mcm1957 commented Mar 9, 2025

README.md changelog added into this PR

@mcm1957 mcm1957 requested a review from foxriver76 March 9, 2025 23:01
Copy link
Contributor

@GermanBluefox GermanBluefox left a comment

Choose a reason for hiding this comment

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

Lgtm

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.

3 participants