Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix package main resolving issue #5344

Merged
merged 5 commits into from
Jan 19, 2018
Merged

Conversation

kenzable
Copy link
Contributor

Summary

Related issue is #5088

Test plan

No additional tests added.
Ran existing tests with all passing.
Manually tested on separate repo that exhibited the original issue and it was successfully resolved.

@codecov-io
Copy link

codecov-io commented Jan 18, 2018

Codecov Report

Merging #5344 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5344   +/-   ##
=======================================
  Coverage   61.32%   61.32%           
=======================================
  Files         205      205           
  Lines        6909     6909           
  Branches        3        3           
=======================================
  Hits         4237     4237           
  Misses       2671     2671           
  Partials        1        1
Impacted Files Coverage Δ
packages/jest-resolve/src/default_resolver.js 96.77% <100%> (ø) ⬆️

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 f040725...f23c379. Read the comment docs.

@cpojer
Copy link
Member

cpojer commented Jan 18, 2018

Do you mind adding an integration test for this behavior?

@cpojer
Copy link
Member

cpojer commented Jan 18, 2018

Also, thanks for the PR!

@eaviles
Copy link

eaviles commented Jan 18, 2018

Thanks @kenzable!

@kenzable
Copy link
Contributor Author

Added some tests @cpojer

Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

Thanks! Clean fix 🙂

@cpojer cpojer merged commit 5943dc9 into jestjs:master Jan 19, 2018
@cpojer
Copy link
Member

cpojer commented Jan 19, 2018

Awesome! Thank you.

@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 12, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants