Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Sign in
Toggle navigation
H
heroku-buildpack-nodejs
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
JIRA
JIRA
Merge Requests
0
Merge Requests
0
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Commits
Issue Boards
Open sidebar
Python-Dev
heroku-buildpack-nodejs
Commits
58457e07
Commit
58457e07
authored
May 13, 2016
by
Hunter Loftis
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #313 from heroku/v90-no-prune
remove npm prune to work around shrinkwrap issues
parents
23811086
4b7f961f
Changes
3
Show whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
1 addition
and
3 deletions
+1
-3
CHANGELOG.md
CHANGELOG.md
+1
-0
dependencies.sh
lib/dependencies.sh
+0
-2
run
test/run
+0
-1
No files found.
CHANGELOG.md
View file @
58457e07
...
...
@@ -3,6 +3,7 @@
## Current
-
Do not fail build on output errors
-
Do not prune before install (shrinkwrap unsupported by prune)
## v90 (2016-4-20)
...
...
lib/dependencies.sh
View file @
58457e07
...
...
@@ -13,8 +13,6 @@ install_node_modules() {
if
[
-e
$build_dir
/package.json
]
;
then
cd
$build_dir
echo
"Pruning any extraneous modules"
npm prune
--unsafe-perm
--userconfig
$build_dir
/.npmrc 2>&1
if
[
-e
$build_dir
/npm-shrinkwrap.json
]
;
then
echo
"Installing node modules (package.json + shrinkwrap)"
else
...
...
test/run
View file @
58457e07
...
...
@@ -214,7 +214,6 @@ testModulesCheckedIn() {
assertCaptured
"(preinstall script)"
assertCaptured
"Installing any new modules"
assertCaptured
"(postinstall script)"
assertNotCaptured
"Pruning any extraneous modules"
assertCapturedSuccess
}
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment