Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Update @typescript-eslint to v6 #325

Merged
merged 3 commits into from
Jul 13, 2023

Conversation

dartess
Copy link
Contributor

@dartess dartess commented Jul 11, 2023

@iamturns hey!

Sorry for the notice. typescript-eslint recently released version 6 with a large set of changes: https://typescript-eslint.io/blog/announcing-typescript-eslint-v6/ . Unfortunately, without changing the peer-deps of your package, we cannot upgrade.

Also i removed those dependencies from dev-deps. It looks like they are not involved in any way during the development of the package. If i am wrong please tell me and i will get back and update them. It was reverted.

I tested the package locally in my codebase and didn't find any problems.

I will be grateful for the review and release of the new version. I don't see any visible changes to how this package works, so this shouldn't be a breaking change for users.

Closed #324

@scott-avery
Copy link

@iamturns could you review this and merge?

@iamturns iamturns merged commit 8ef77c9 into iamturns:master Jul 13, 2023
@iamturns
Copy link
Owner

Thanks @dartess :)

@Methuselah96
Copy link

@iamturns Looks like the CI release failed.

@ghost
Copy link

ghost commented Jul 13, 2023

Hi @iamturns , when will this be published on NPM?

@iamturns
Copy link
Owner

Looking into it now, some tokens have expired

Hopefully @typescript-eslint v5 is ok to use for a bit longer!

@iamturns
Copy link
Owner

🎉 This PR is included in version 17.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ghost
Copy link

ghost commented Jul 13, 2023

Great 😃

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

need to update dependencies
6 participants