Skip to content

fix(unicorn/linter) fix multiple false positives in no-useless-promise-resolve-reject#5178

Merged
graphite-app[bot] merged 1 commit intomainfrom
c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext
Aug 24, 2024
Merged

fix(unicorn/linter) fix multiple false positives in no-useless-promise-resolve-reject#5178
graphite-app[bot] merged 1 commit intomainfrom
c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext

Conversation

@camc314
Copy link
Copy Markdown
Contributor

@camc314 camc314 commented Aug 24, 2024

more closely follows the implementation from unicorn's repo.

early returns if the parent is not a return, yield, or an arroy function

Closes #5125
Closes #5126

@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app bot commented Aug 24, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@github-actions github-actions bot added the A-linter Area - Linter label Aug 24, 2024
@camc314 camc314 marked this pull request as ready for review August 24, 2024 20:04
Copy link
Copy Markdown
Contributor Author

camc314 commented Aug 24, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camc314 and the rest of your teammates on Graphite Graphite

@camc314 camc314 force-pushed the c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext branch from 33ded8c to 6d0b998 Compare August 24, 2024 20:05
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Aug 24, 2024

CodSpeed Performance Report

Merging #5178 will not alter performance

Comparing c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext (dc9e1e2) with main (d29042e)

Summary

✅ 29 untouched benchmarks

@camc314 camc314 changed the title fix(unicorn/linter) fix multiple false positives in `no-useless-promise-resolve-rejext fix(unicorn/linter) fix multiple false positives in no-useless-promise-resolve-reject Aug 24, 2024
@DonIsaac DonIsaac added the 0-merge Merge with Graphite Merge Queue label Aug 24, 2024
@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app bot commented Aug 24, 2024

Merge activity

  • Aug 24, 5:03 PM EDT: The merge label 'merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Aug 24, 5:03 PM EDT: DonIsaac added this pull request to the Graphite merge queue.
  • Aug 24, 5:05 PM EDT: DonIsaac merged this pull request with the Graphite merge queue.

…se-resolve-reject` (#5178)

more closely follows the implementation from unicorn's repo.

early returns if the parent is not a `return`, `yield`, or an arroy function

Closes #5125
Closes #5126
@DonIsaac DonIsaac force-pushed the c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext branch from 6d0b998 to dc9e1e2 Compare August 24, 2024 21:03
@graphite-app graphite-app bot merged commit dc9e1e2 into main Aug 24, 2024
@graphite-app graphite-app bot deleted the c/08-24-fix_unicorn_linter_fix_multiple_false_positives_in_no-useless-promise-resolve-rejext branch August 24, 2024 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-linter Area - Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(linter) bug in fixer for no-useless-promise-resolve-reject bug(linter) bug in fixer for no-useless-promise-resolve-reject

2 participants