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

Avoid destructuring and just build the expected object once #538

Merged
merged 2 commits into from
May 9, 2021

Conversation

fregante
Copy link
Member

@fregante fregante commented May 9, 2021

Tiny changes of code I noticed in https://github.com/xojs/xo/pull/534/files

@codecov-commenter
Copy link

Codecov Report

Merging #538 (31068d4) into main (4cfdc72) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #538      +/-   ##
==========================================
- Coverage   97.05%   97.02%   -0.04%     
==========================================
  Files           8        8              
  Lines         442      437       -5     
==========================================
- Hits          429      424       -5     
  Misses         13       13              
Impacted Files Coverage Δ
index.js 98.79% <100.00%> (-0.07%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4cfdc72...31068d4. Read the comment docs.

@sindresorhus
Copy link
Member

@Richienb Can you review?

Copy link
Contributor

@Richienb Richienb left a comment

Choose a reason for hiding this comment

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

Looks about right.

@sindresorhus sindresorhus changed the title Avoid destructuring and just build the expected object once Avoid destructuring and just build the expected object once May 9, 2021
@sindresorhus sindresorhus merged commit 8e1801c into xojs:main May 9, 2021
@fregante fregante deleted the patch-1 branch May 9, 2021 14:20
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.

4 participants