Skip to content

fix: Move @types/estree to devDependencies#72

Closed
timfish wants to merge 1 commit into
mainfrom
fix/dev-dep
Closed

fix: Move @types/estree to devDependencies#72
timfish wants to merge 1 commit into
mainfrom
fix/dev-dep

Conversation

@timfish
Copy link
Copy Markdown
Contributor

@timfish timfish commented May 20, 2026

This is only required at dev time.

@timfish timfish requested a review from rochdev May 21, 2026 07:38
@rochdev
Copy link
Copy Markdown
Contributor

rochdev commented May 21, 2026

This is only required at dev time.

Is it though? Since index.d.ts references it, if it's not installed it might break TypeScript for users.

@timfish
Copy link
Copy Markdown
Contributor Author

timfish commented May 22, 2026

You're right, I missed that detail!

@timfish timfish closed this May 22, 2026
@jsumners-nr jsumners-nr deleted the fix/dev-dep branch May 26, 2026 11:44
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