Skip to content
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

feat: disallow awaiting non thenable values #110

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

mpetrunic
Copy link
Contributor

@mpetrunic mpetrunic commented Jul 14, 2022

add promise related lint rules

@mpetrunic mpetrunic changed the title Update ts.js Update ts.js with promise related rules Jul 14, 2022
@mpetrunic mpetrunic changed the title Update ts.js with promise related rules feat: update ts.js with promise related rules Jul 14, 2022
@mpetrunic
Copy link
Contributor Author

@wemeetagain @achingbrain

@BigLep BigLep requested a review from achingbrain July 22, 2022 14:07
ts.js Outdated Show resolved Hide resolved
ts.js Outdated Show resolved Hide resolved
ts.js Outdated Show resolved Hide resolved
@achingbrain achingbrain changed the title feat: update ts.js with promise related rules feat!: update ts.js with promise related rules Aug 10, 2022
@mpetrunic mpetrunic changed the title feat!: update ts.js with promise related rules feat!: disallow awaiting non thenable values Aug 10, 2022
@achingbrain achingbrain changed the title feat!: disallow awaiting non thenable values feat: disallow awaiting non thenable values Aug 30, 2022
@achingbrain achingbrain merged commit 602fa7e into ipfs:default Aug 30, 2022
achingbrain added a commit to ipfs/aegir that referenced this pull request Jan 12, 2023
Updates all deps to the latest versions. Includes new linting rules from
`eslint-config-ipfs` that require return types on all methods to prevent
tsc creating verbose derived types when we normally meant to specify
that a function returns an interface.

- ipfs/eslint-config-ipfs#114
- ipfs/eslint-config-ipfs#110

BREAKING CHANGE: all functions and methods need to declare a return type
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