search

hoodiehq-archive / my-first-hoodie

⛔ deprecated
Other
157 stars 35 forks source link

Shrinkwrap hoodie apps by default #72

Closed boennemann closed 9 years ago

boennemann commented 9 years ago

For discussion, don't merge yet.

Currently exploring if this installs right, and if we need to adapt the release process for this.

boennemann commented 9 years ago

Okay on first sight it does not work out of the box, because of some custom fiddling with the package.json and cleanups, that remove the shrinkwrap file entirely. We need to build shrinkwrap support into hoodie-cli first.

We also need to check if hoodie install works right for plugins then.

svnlto commented 9 years ago

was gonna say that. hoodie-cli does a bunch of file removals (legacy). So this won't work out of the box.

boennemann commented 9 years ago

Actually there was some caching going on during my first try. The cleanup thingy happened with git, but the cli is now based on npm.

This works right now, but the name field in the npm-shrinkwrap.json isn't updated. Here is a PR to fix that in the CLI https://github.com/hoodiehq/hoodie-cli/pull/160