Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
feat(eslint-plugin): [no-invalid-void-type] better report message for void used as a constituent inside a function return type#5274
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
fix examples and --Isolatedmodules explain
one is correct, another is incorrect
kmin-jeong commentedAug 18, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@JoshuaKGoldberg@bradzacher |
Nope, PRs generally stay open unless some drastic difference in strategy is decided upon (not likely here) or they aren't updated by the author in months. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
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.
Let me know if you need help with any of this 🙁. I get the impression this is a lot of new stuff - feel free to ask questions on anything that doesn't make sense!
https://sourcegraph.com/notebooks/Tm90ZWJvb2s6MTA2OA== has some good docs on how our rules work, if that helps.
Ping@kmin-jeong, do you still have time to work on this? No worries if not, just want to make sure it's not pending too long. 🙂 |
@JoshuaKGoldberg I don't have enough time to solve this , so can I hand it over to someone else? |
No worries, thanks for getting this started! I'll take it over right now. 😄 |
This PR contains the following updates:| Package | Type | Update | Change ||---|---|---|---|| [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint) | devDependencies | minor | [`5.41.0` -> `5.42.0`](https://renovatebot.com/diffs/npm/@typescript-eslint%2feslint-plugin/5.41.0/5.42.0) || [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint) | devDependencies | minor | [`5.41.0` -> `5.42.0`](https://renovatebot.com/diffs/npm/@typescript-eslint%2fparser/5.41.0/5.42.0) |---### Release Notes<details><summary>typescript-eslint/typescript-eslint (@​typescript-eslint/eslint-plugin)</summary>### [`v5.42.0`](https://github.com/typescript-eslint/typescript-eslint/blob/HEAD/packages/eslint-plugin/CHANGELOG.md#​5420-httpsgithubcomtypescript-eslinttypescript-eslintcomparev5410v5420-2022-10-31)[Compare Source](typescript-eslint/typescript-eslint@v5.41.0...v5.42.0)##### Bug Fixes- **ast-spec:** add TSQualifiedName to TypeNode union ([#​5906](typescript-eslint/typescript-eslint#5906)) ([5c316c1](typescript-eslint/typescript-eslint@5c316c1))- **eslint-plugin:** \[no-extra-parens] handle type assertion in extends clause ([#​5901](typescript-eslint/typescript-eslint#5901)) ([8ed7219](typescript-eslint/typescript-eslint@8ed7219))##### Features- **eslint-plugin:** \[member-ordering] add natural sort order ([#​5662](typescript-eslint/typescript-eslint#5662)) ([1eaae09](typescript-eslint/typescript-eslint@1eaae09))- **eslint-plugin:** \[no-invalid-void-type] better report message for void used as a constituent inside a function return type ([#​5274](typescript-eslint/typescript-eslint#5274)) ([d806bda](typescript-eslint/typescript-eslint@d806bda))</details><details><summary>typescript-eslint/typescript-eslint (@​typescript-eslint/parser)</summary>### [`v5.42.0`](https://github.com/typescript-eslint/typescript-eslint/blob/HEAD/packages/parser/CHANGELOG.md#​5420-httpsgithubcomtypescript-eslinttypescript-eslintcomparev5410v5420-2022-10-31)[Compare Source](typescript-eslint/typescript-eslint@v5.41.0...v5.42.0)##### Features- **scope-manager:** ignore ECMA version ([#​5881](typescript-eslint/typescript-eslint#5881)) ([3b8d449](typescript-eslint/typescript-eslint@3b8d449))##### Reverts- Revert "feat(scope-manager): ignore ECMA version" ([#​5888](typescript-eslint/typescript-eslint#5888)) ([2ee81df](typescript-eslint/typescript-eslint@2ee81df)), closes [#​5888](typescript-eslint/typescript-eslint#5888) [#​5881](typescript-eslint/typescript-eslint#5881)</details>---### Configuration📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.--- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box---This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC45LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNC4xMC4wIn0=-->Co-authored-by: cabr2-bot <cabr2.help@gmail.com>Reviewed-on:https://codeberg.org/Calciumdibromid/CaBr2/pulls/1619Reviewed-by: Epsilon_02 <epsilon_02@noreply.codeberg.org>Co-authored-by: Calciumdibromid Bot <cabr2_bot@noreply.codeberg.org>Co-committed-by: Calciumdibromid Bot <cabr2_bot@noreply.codeberg.org>
Uh oh!
There was an error while loading.Please reload this page.
PR Checklist
Overview
change and error message in case of union type