Skip to content

Commit

Permalink
build the icons when needed
Browse files Browse the repository at this point in the history
It seems the prepare script has some issues and we can't always depend on it for donig the right thing. We'll just run the build script when needed.
  • Loading branch information
Windvis committed Mar 21, 2024
1 parent 21851f3 commit 6cf23ae
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ jobs:
cache: npm
- name: Install Dependencies
run: npm ci
- name: Build icons
run: npm run build:icons
- name: Lint
run: npm run lint
- name: Run Tests
Expand All @@ -44,6 +46,8 @@ jobs:
cache: npm
- name: Install Dependencies
run: npm install --no-shrinkwrap
- name: Build icons
run: npm run build:icons
- name: Run Tests
run: npm run test:ember

Expand Down Expand Up @@ -73,5 +77,7 @@ jobs:
cache: npm
- name: Install Dependencies
run: npm ci
- name: Build icons
run: npm run build:icons
- name: Run Tests
run: ./node_modules/.bin/ember try:one ${{ matrix.try-scenario }}
14 changes: 6 additions & 8 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@
"test": "tests"
},
"scripts": {
"build": "run-s icons build-ember build-storybook",
"build:icons": "node lib/generate-icon-components.mjs",
"build": "run-s build-ember build-storybook",
"build:icons": "concurrently \"node lib/generate-icon-components.mjs\" \"svg-symbols ./public/icons > ./public/appuniversum-symbolset.svg\"",
"build:types": "glint -d",
"lint": "concurrently \"npm:lint:*(!fix)\" --names \"lint:\"",
"lint:css": "stylelint --allow-empty-input \"**/*.css\"",
Expand All @@ -46,20 +46,18 @@
"lint:js": "eslint . --cache",
"lint:js:fix": "eslint . --fix",
"lint:types": "glint",
"prepack": "glint -d",
"prepack": "run-s build:icons build:types",
"postpack": "rimraf declarations",
"start": "run-p ember storybook",
"ember": "ember serve",
"ember": "npm run build:icons && ember serve",
"test": "concurrently \"npm:lint\" \"npm:test:*\" --names \"lint,test:\"",
"test:ember": "ember test",
"test:ember-compatibility": "ember try:each",
"icons": "svg-symbols ./public/icons > ./public/appuniversum-symbolset.svg",
"prepare": "concurrently npm:build:icons npm:icons",
"storybook": "start-storybook -p 6006",
"build-storybook": "build-storybook",
"build-ember": "ember build --environment=production",
"build-ember": "npm run build:icons && ember build --environment=production",
"build-ember-chromatic": "ember build --environment=chromatic",
"prep-chromatic": "run-s icons build-ember-chromatic",
"prep-chromatic": "run-s build:icons build-ember-chromatic",
"release": "release-it",
"chromatic": "chromatic --exit-zero-on-changes"
},
Expand Down

0 comments on commit 6cf23ae

Please sign in to comment.