Skip to content

Conversation

@rekmarks
Copy link
Member

Explanation

Following the recommendation of @mcmire, converts @types/eslint from a dev to production dependency of @metamask/build-utils.

The reason:

ESLint will show up in the type definition file for src/transforms/utils.ts, so anyone who uses this library in a TypeScript will need that type in order to compile their code.

References

Changelog

N/A (package unreleased)

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@rekmarks rekmarks requested a review from a team as a code owner November 29, 2023 19:28
@rekmarks rekmarks requested a review from mcmire November 29, 2023 19:38
Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Looks good!

@rekmarks rekmarks merged commit 30b69ff into main Nov 30, 2023
@rekmarks rekmarks deleted the update-build-utils-type-deps branch November 30, 2023 01:12
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