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

refactor: use is prefix for booleans #4674

Merged
merged 4 commits into from Jan 6, 2019
Merged

Conversation

manniL
Copy link
Member

@manniL manniL commented Jan 3, 2019

See the small remark in #4671

Types of changes

  • Refactor

Checklist:

  • All new and existing tests are passing.

@manniL manniL requested a review from pi0 January 3, 2019 00:44
@codecov-io
Copy link

codecov-io commented Jan 3, 2019

Codecov Report

Merging #4674 into dev will decrease coverage by 0.14%.
The diff coverage is 63.63%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #4674      +/-   ##
==========================================
- Coverage   90.59%   90.45%   -0.15%     
==========================================
  Files          67       67              
  Lines        2244     2252       +8     
  Branches      551      555       +4     
==========================================
+ Hits         2033     2037       +4     
- Misses        190      194       +4     
  Partials       21       21
Impacted Files Coverage Δ
packages/core/src/resolver.js 83.87% <63.63%> (-5.02%) ⬇️

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 d063cb0...d2a1f89. Read the comment docs.

pi0
pi0 previously approved these changes Jan 3, 2019
Copy link
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

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

LGTM. Just left some personal toughts

packages/core/src/resolver.js Show resolved Hide resolved
packages/core/src/resolver.js Outdated Show resolved Hide resolved
packages/core/src/resolver.js Outdated Show resolved Hide resolved
galvez
galvez previously approved these changes Jan 3, 2019
@manniL manniL dismissed stale reviews from galvez and pi0 via 1ed9e90 January 5, 2019 21:59
@pi0 pi0 merged commit 78433b5 into dev Jan 6, 2019
@pi0 pi0 deleted the refactor-use-is-prefix-for-booleans branch January 6, 2019 08:02
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants