Skip to content
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.

Add postinstall scripts which throw an error if the os or cpu is different to what the package supports #4

Merged
merged 1 commit into from
Mar 13, 2020

Conversation

JakeChampion
Copy link
Contributor

This is a required for npm ci to work. npm/cli#558

@JakeChampion JakeChampion requested a review from a team as a code owner March 11, 2020 13:45
…erent to what the package supports

     This is a required for npm ci to work. npm/cli#558
@JakeChampion
Copy link
Contributor Author

@Financial-Times/origami-core Each package has the same change, a postinstall.js file which is exactly the same and a postinstall npm run-script which runts the postinstall.js file. You only need to review one file to https://github.com/Financial-Times/sass/pull/4/files#diff-cb76c36cc058782c5856812a25bcaa5e :-)

@Financial-Times Financial-Times deleted a comment from wheresrhys Mar 12, 2020
Copy link
Contributor

@notlee notlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wild 😊

@JakeChampion JakeChampion merged commit f253841 into master Mar 13, 2020
@JakeChampion JakeChampion deleted the build branch March 13, 2020 10:33
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.

2 participants