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.

Sign up for GitHub

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

Jump to bottom

fix: helpful error message when attempting to use the package main file #29

Merged
merged 2 commits into from May 5, 2019
Merged

fix: helpful error message when attempting to use the package main file #29

merged 2 commits into from May 5, 2019

Conversation

jeysal
Copy link
Member

@jeysal jeysal commented May 4, 2019

Fixes #27
image

@SimenB
Copy link
Member

SimenB commented May 5, 2019

Perfect! Follow-up PR to jest fixing the error so it gets formatted more like other config errors?

@SimenB SimenB merged commit 716fe42 into jest-community:master May 5, 2019
@jeysal jeysal mentioned this pull request May 5, 2019
Merged
@jeysal
Copy link
Member Author

jeysal commented May 5, 2019

Perfect! Follow-up PR to jest fixing the error so it gets formatted more like other config errors?

Great idea! jestjs/jest#8422
Having some problems getting it to print nicely though

@jeysal jeysal deleted the main-file branch May 5, 2019 22:28
@snyk-bot snyk-bot mentioned this pull request Feb 25, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimenB SimenB SimenB approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Main entry point build/index.js does not exist
2 participants
@jeysal @SimenB