Skip to content

[Fix] no-import-module-exports: Don't crash if packages have no entrypoint#2099

Merged
ljharb merged 1 commit into
import-js:masterfrom
eps1lon:fix/no-import-module-exports/missing-entrypoint
May 26, 2021
Merged

[Fix] no-import-module-exports: Don't crash if packages have no entrypoint#2099
ljharb merged 1 commit into
import-js:masterfrom
eps1lon:fix/no-import-module-exports/missing-entrypoint

Conversation

@eps1lon

@eps1lon eps1lon commented May 26, 2021

Copy link
Copy Markdown
Contributor

getEntryPoint assumes every package has one. However, a package does not need to have an entry point e.g. CLI packages such as kcd-scripts.

Fixes kentcdodds/kcd-scripts#209

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 82.462% when pulling a291621 on eps1lon:fix/no-import-module-exports/missing-entrypoint into 20c373c on benmosher:master.

@coveralls

coveralls commented May 26, 2021

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.009%) to 82.502% when pulling 81b9d24 on eps1lon:fix/no-import-module-exports/missing-entrypoint into 20c373c on benmosher:master.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 82.462% when pulling a291621 on eps1lon:fix/no-import-module-exports/missing-entrypoint into 20c373c on benmosher:master.

@ljharb
ljharb force-pushed the fix/no-import-module-exports/missing-entrypoint branch from a291621 to 81b9d24 Compare May 26, 2021 16:22
@ljharb ljharb changed the title fix(no-import-module-exports): Don't crash if packages have no entry point [Fix] no-import-module-exports: Don't crash if packages have no entrypoint May 26, 2021
@ljharb ljharb changed the title [Fix] no-import-module-exports: Don't crash if packages have no entrypoint [Fix] no-import-module-exports: Don't crash if packages have no entrypoint May 26, 2021
@ljharb
ljharb merged commit 81b9d24 into import-js:master May 26, 2021
@eps1lon
eps1lon deleted the fix/no-import-module-exports/missing-entrypoint branch May 26, 2021 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

fix linting issues on windows

3 participants