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

Make /lib in deep import paths optional #926

Closed
wants to merge 2 commits into from
Closed

Conversation

slikts
Copy link

@slikts slikts commented Aug 8, 2019

Enables importing modules as import { Either } from 'fp-ts/Either' instead of 'fp-ts/lib/Either' while staying backward compatible. It publishes from /lib instead of the package root, copies and patches package.json to /lib, and copies all generated source files to /lib/lib to keep backward compatibility.

The changes are intended to be minimal, but one caveat is that npm-link also needs to run from the /lib directory to work, and that changes in the copied files (/lib/lib and /lib/package.json) won't be updated by using tsc watch mode.

@gcanti
Copy link
Owner

gcanti commented Jan 11, 2020

Closing in favor of #1053

@gcanti gcanti closed this Jan 11, 2020
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.

2 participants