Skip to content
This repository has been archived by the owner on Mar 22, 2024. It is now read-only.

Commit

Permalink
Don't run npm scripts in integration-tests
Browse files Browse the repository at this point in the history
This will prevent the check-engines script throwing an error in
next-service-registry integration test when we are testing with Node 14
in the CI. Hopefully won't cause unintended side-effects from skipping
other installation scripts (particularly `prepare`.)
  • Loading branch information
ivomurrell committed Mar 18, 2022
1 parent 782951e commit 8585f6b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ integration-test-%: $(fixture-base)/%
sed -i'.bak' "s:-include node_modules/@financial-times/n-gage/index.mk:include $(ngage-path):" $</Makefile

# run a handful of standard next make commands in the fixture folder using the `-C DIRECTORY` option
$(MAKE) -C $< install build
npm_config_ignore_scripts=true $(MAKE) -C $< install build

# ensure the folders have some basic things we would expect from install
[ -d $</node_modules ]
Expand Down

0 comments on commit 8585f6b

Please sign in to comment.