Actions
Bug #33319
closeddon't run npm-fix-foreman-stories.sh on every npm install
Difficulty:
Triaged:
No
Description
Right now, we run npm-fix-foreman-stories.sh in postinstall, so on every "npm install" in Foreman, this gets executed and pulls in thousands of deps.
The script guards by $NODE_ENV != "production", but when building assets for packaging, we don't set NODE_ENV to "production" (probably because we need devDependencies actually installed as it contains webpack etc?)
So my questions would be:
1. can we just move npm-fix-foreman-stories.sh to a step that is not automatically executed on every "npm i"?
2. should we run package builds with NODE_ENV=production?
3. what about CI? are storybooks needed there? my gut is saying no?
Updated by Evgeni Golov over 3 years ago
- Related to Bug #33317: better differentiate between build, develop and test dependencies for JavaScript added
Updated by The Foreman Bot over 3 years ago
- Status changed from New to Ready For Testing
- Assignee set to Evgeni Golov
- Pull request https://github.com/theforeman/foreman/pull/8730 added
Updated by Evgeni Golov over 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman|2c96294a124d5f6e13d48fc1841fb8441a648e18.
Updated by The Foreman Bot over 3 years ago
- Pull request https://github.com/theforeman/foreman/pull/8736 added
Updated by Evgeni Golov over 3 years ago
- Fixed in Releases 3.0.0 added
- Fixed in Releases deleted (
3.1.0)
Actions