Skip to content

test(eslint-plugin): [unified-signatures] add tests for #740 #1074

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

Merged

Conversation

komkanit
Copy link
Contributor

Resolve #740
I try to add async function test case but this bug already solved in some pr.
So I add test case about async function to make sure this issue solved

This is my first pr on typescript-eslint I'm not sure this is collect or not. you can recommend anything. Thank you

@typescript-eslint
Copy link
Contributor

Thanks for the PR, @komcal!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!


🙏 Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitors per day.

@komkanit komkanit force-pushed the unified-signatures-promise-test branch from 6bfe8b8 to 978712f Compare October 12, 2019 15:36
@bradzacher bradzacher changed the title fix(eslint-plugin): [unified-sugnatures] add async function test case fix(eslint-plugin): [unified-signatures] add async function test case Oct 12, 2019
@bradzacher bradzacher added the tests anything to do with testing label Oct 12, 2019
@bradzacher bradzacher changed the title fix(eslint-plugin): [unified-signatures] add async function test case chore(eslint-plugin): [unified-signatures] add async function test case Oct 12, 2019
@codecov
Copy link

codecov bot commented Dec 3, 2019

Codecov Report

Merging #1074 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1074      +/-   ##
==========================================
+ Coverage   94.01%   94.06%   +0.04%     
==========================================
  Files         131      115      -16     
  Lines        5797     5086     -711     
  Branches     1638     1423     -215     
==========================================
- Hits         5450     4784     -666     
+ Misses        186      174      -12     
+ Partials      161      128      -33
Impacted Files Coverage Δ
packages/eslint-plugin/src/rules/camelcase.ts 82.14% <0%> (-10.72%) ⬇️
packages/eslint-plugin/src/rules/require-await.ts 92.68% <0%> (-7.32%) ⬇️
packages/typescript-estree/src/convert-comments.ts 96.29% <0%> (-3.71%) ⬇️
.../eslint-plugin/src/rules/restrict-plus-operands.ts 97.22% <0%> (-2.78%) ⬇️
packages/typescript-estree/src/parser.ts 91.89% <0%> (-2.18%) ⬇️
...ckages/eslint-plugin/src/rules/no-magic-numbers.ts 92.5% <0%> (-1.84%) ⬇️
packages/eslint-plugin/src/rules/array-type.ts 92.63% <0%> (-0.71%) ⬇️
packages/eslint-plugin/src/rules/no-type-alias.ts 94.54% <0%> (-0.54%) ⬇️
packages/eslint-plugin/src/rules/indent.ts 92.3% <0%> (-0.38%) ⬇️
...-plugin/src/rules/no-unnecessary-type-arguments.ts 91.66% <0%> (-0.34%) ⬇️
... and 56 more

bradzacher
bradzacher previously approved these changes Dec 3, 2019
Copy link
Member

@bradzacher bradzacher left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for this.

@bradzacher bradzacher changed the title chore(eslint-plugin): [unified-signatures] add async function test case test(eslint-plugin): [unified-signatures] add tests for #740 Dec 3, 2019
@bradzacher bradzacher merged commit 39f654b into typescript-eslint:master Dec 3, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests anything to do with testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[unified-signatures] Rule is confused by a difference in signatures on an async function's return type
2 participants