-
Notifications
You must be signed in to change notification settings - Fork 364
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
chore: update eslint-config-ipfs and fix lint #413
Conversation
SgtPooki
commented
Jun 8, 2023
•
edited
Loading
edited
- Update eslint-config-ipfs.
- Moved app-specific types to types.ts from global.d.ts due to import() type eslint rule violations.
- add type imports where types are needed and used to be globally available.
66b84db
to
6c2f99e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this somewhat overlaps with #311
Can you please fill out the description for this issue and what's not covered? Also some nits.
I don't think it does at all, since this only involves fixing linting issues.. there is zero logic change in this, and practically zero code changes that aren't only moving things from global.d.ts -> types.ts and adding type imports where things used to be globally available.
Updated description |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix nits, and 🚀
## [1.2.2](v1.2.1...v1.2.2) (2023-06-08) ### Trivial Changes * update eslint-config-ipfs and fix lint ([#413](#413)) ([165f35c](165f35c))
🎉 This PR is included in version 1.2.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |