Update ESLint configuration and packages#118
Conversation
The ESLint configuration and all related dependencies have been updated. Most changes are related to the JSDoc lint rules; many JSDoc blocks were added or adjusted. All other lint changes were applied automatically using `yarn lint:fix`.
| and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).`; | ||
|
|
||
| interface ReleaseMetadata { | ||
| type ReleaseMetadata = { |
There was a problem hiding this comment.
I've seen this change made in a few commits and I'm curious about the motivation behind the shift from interface to type. Is this something we are slowly changing across the project, or is there a specific need that we are addressing?
There was a problem hiding this comment.
See here for more context: MetaMask/eslint-config#216 (and this Slack thread for even more context)
Primarily it was for consistency. No specific need being addressed. Erik referenced some difficulties encountered working with interfaces and our Json type, but I can't recall the details on that.
There was a problem hiding this comment.
This is great, thanks for the context @Gudahtt!
There was a problem hiding this comment.
Erik referenced some difficulties encountered working with interfaces and our
Jsontype, but I can't recall the details on that.
It has to do with interfaces not having an index signature, IIRC.
Mrtenz
left a comment
There was a problem hiding this comment.
Looks good overall. Just some nits.
The ESLint configuration and all related dependencies have been updated. Most changes are related to the JSDoc lint rules; many JSDoc blocks were added or adjusted. All other lint changes were applied automatically using
yarn lint:fix.