Skip to content

Fix demo build CI #99

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
b-yogesh merged 2 commits into from
Jan 14, 2025
Merged

Fix demo build CI #99

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b861214
Update frontend-ci.yml
b-yogesh Jan 14, 2025
5c78569
Update publish-frontend.yml
b-yogesh Jan 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 36 additions & 20 deletions .github/workflows/frontend-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,25 +49,41 @@ jobs:

npm-tests-demo:
runs-on: ubuntu-latest
needs: npm-tests-lib
strategy:
matrix:
node-version: [18.x, 20.x]
# See supported Node.js release schedule at
# https://nodejs.org/en/about/releases/
defaults:
run:
working-directory: chartlets.js/packages/demo

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
- name: git-checkout chartlets
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'
cache-dependency-path: chartlets.js/package-lock.json

- run: npm ci
- run: npm run lint
- run: npm run build
node-version: '20.x'

- name: Install common dependencies
run: |
cd chartlets.js
npm install

- name: Lib lint
run: |
cd chartlets.js/packages/lib
npm run lint

- name: Lib tests
run: |
cd chartlets.js/packages/lib
npm run test

- name: Lib build
run: |
cd chartlets.js/packages/lib
npm run build

- name: Demo lint
run: |
cd chartlets.js/packages/demo
npm run lint

- name: Demo build
run: |
cd chartlets.js/packages/demo
npm run build
56 changes: 36 additions & 20 deletions .github/workflows/publish-frontend.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,28 +44,44 @@ jobs:

npm-tests-demo:
runs-on: ubuntu-latest
needs: npm-tests-lib
strategy:
matrix:
node-version: [18.x, 20.x]
# See supported Node.js release schedule at
# https://nodejs.org/en/about/releases/
defaults:
run:
working-directory: chartlets.js/packages/demo

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
- name: git-checkout chartlets
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'
cache-dependency-path: chartlets.js/package-lock.json

- run: npm ci
- run: npm run lint
- run: npm run build
node-version: '20.x'

- name: Install common dependencies
run: |
cd chartlets.js
npm install

- name: Lib lint
run: |
cd chartlets.js/packages/lib
npm run lint

- name: Lib tests
run: |
cd chartlets.js/packages/lib
npm run test

- name: Lib build
run: |
cd chartlets.js/packages/lib
npm run build

- name: Demo lint
run: |
cd chartlets.js/packages/demo
npm run lint

- name: Demo build
run: |
cd chartlets.js/packages/demo
npm run build

npm-deploy:
name: Publish TS-React Package to npmjs
Expand Down