Skip to content
This repository was archived by the owner on Apr 18, 2020. It is now read-only.

Feature/buildability - #8

Merged
bajtos merged 3 commits into
masterfrom
feature/buildability
Jun 3, 2014
Merged

Feature/buildability#8
bajtos merged 3 commits into
masterfrom
feature/buildability

Conversation

@sam-github

Copy link
Copy Markdown
Contributor

strong-build support

/to @ritch

Supervisor needs a main property in the package to find what to run if
there is no top-level server.js or app.js.
strong-build uses a build script (if it exists) to run custom build
scripts before call `npm pack`.
Not strictly necessary for strong-build, but having one is safer and
more explicit.  I also sorted .gitignore, to allow a simple diff of
.gitignore and .npmignore to verify that build products are packed and
non-build files (such as editor ephemera, test output, npm log files,
etc.) are still ignored by npm.
@sam-github

Copy link
Copy Markdown
Contributor Author

/cc @bajtos

@bajtos

bajtos commented Jun 3, 2014

Copy link
Copy Markdown
Member

LGTM. If @ritch does not object by EOD Tuesday, I'll merge land this patch.

@ritch

ritch commented Jun 3, 2014

Copy link
Copy Markdown
Member

👍

bajtos added a commit that referenced this pull request Jun 3, 2014
@bajtos
bajtos merged commit e41616e into master Jun 3, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants