Skip to content
This repository has been archived by the owner on Sep 4, 2019. It is now read-only.

npm package is broken. #12

Closed
marshallswain opened this issue Nov 26, 2015 · 4 comments · Fixed by #13
Closed

npm package is broken. #12

marshallswain opened this issue Nov 26, 2015 · 4 comments · Fixed by #13

Comments

@marshallswain
Copy link
Member

0.3.0 on npm still has a dist folder instead of the src and lib folders that the npm scripts are looking for.

screen shot 2015-11-25 at 9 01 43 pm

@daffl Can you add me to the npm package?
@daffl
Copy link
Member

daffl commented Nov 26, 2015

Already just did for #11. dist/ should be there for Browser clients though. I'm not sure why lib isn't being published though.

@daffl
Copy link
Member

daffl commented Nov 26, 2015

I bet it's because the prepublish script ("prepublish": "npm run compile") is missing.

@daffl daffl mentioned this issue Nov 26, 2015
Closed
@marshallswain
Copy link
Member Author

Yep. That was it.

@marshallswain marshallswain reopened this Nov 26, 2015
@marshallswain
Copy link
Member Author

Opening this back up. It's not working with Steal in its current condition.

screen shot 2015-11-25 at 9 34 00 pm

@marshallswain marshallswain mentioned this issue Nov 26, 2015
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix es6 build and add Steal compatibility. feathersjs-ecosystem/client
2 participants
@marshallswain @daffl