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

Required files are not included in the published npm #630

Closed
shuse2 opened this issue Oct 11, 2018 · 0 comments
Closed

Required files are not included in the published npm #630

shuse2 opened this issue Oct 11, 2018 · 0 comments
Assignees
@shuse2
Labels
🐛bug *easy

Comments

@shuse2
Copy link
Contributor

shuse2 commented Oct 11, 2018

Description

  • default_config.json is not included in the published npm package, and it will throw an error.
  • README.md is overwritten, and it doesn't have linked docs for npm page.

Expected behavior: [What you expect to happen]

  • default_config.json has to be included in the npm package
  • README.md should not be overwritten.
  • docs folder needs to be included in the package for npm page

Versions

v2.0.0-beta.0
@shuse2 shuse2 self-assigned this Oct 11, 2018
@shuse2 shuse2 mentioned this issue Oct 11, 2018
Merged
shuse2 added a commit that referenced this issue Oct 11, 2018
@shuse2
Merge pull request #631 from LiskHQ/630-fix_npm_publish_result
b86484b 
Fix npm publish result - Cloese #630
@shuse2 shuse2 closed this as completed Oct 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@shuse2 shuse2

Labels
🐛bug *easy
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@shuse2