Skip to content
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

Jump to bottom

19384 PNPM replacement #758

Merged
merged 7 commits into from
Apr 24, 2024
Merged

19384 PNPM replacement #758

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c161283
disable Action button when change name to a numbered company and is n…
eve-git Apr 17, 2024
19e3304
Merge branch 'main' into 19384
eve-git Apr 19, 2024
c39c661
pnpm replacement
eve-git Apr 19, 2024
5ea967e
remove Dockerfile and Makefile
eve-git Apr 19, 2024
82e248f
update ci
eve-git Apr 24, 2024
cfa84d4
add build check
eve-git Apr 24, 2024
0b2ea05
resolving conflict
eve-git Apr 24, 2024
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
74 changes: 7 additions & 67 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,77 +3,17 @@ name: NAMEREQUEST UI CI
on:
pull_request:
types: [assigned, synchronize]
workflow_dispatch:

defaults:
run:
shell: bash
working-directory: ./

jobs:
setup-job:
runs-on: ubuntu-20.04

if: github.repository == 'bcgov/namerequest'

steps:
- uses: actions/checkout@v3
- run: "true"

linting:
needs: setup-job
runs-on: ubuntu-20.04

strategy:
matrix:
node-version: [20.5.1]

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node-version }}
- name: Install dependencies
run: |
make setup
- name: Linting
run: |
make lint

testing-coverage:
needs: setup-job
runs-on: ubuntu-20.04

strategy:
matrix:
node-version: [20.5.1]

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node-version }}
- name: Install dependencies
run: |
make setup
- name: Test with Jest
id: test
run: |
make test
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v3
with:
name: codecov-name-request
fail_ci_if_error: true

build-check:
needs: setup-job
runs-on: ubuntu-20.04

steps:
- uses: actions/checkout@v3
- name: build to check strictness
id: build
run: |
make build-nc
namerequest-ci:
uses: bcgov/bcregistry-sre/.github/workflows/frontend-ci.yaml@main
with:
app_name: "namerequest"
working_directory: "./"
codecov_flag: ""
23 changes: 0 additions & 23 deletions Dockerfile
View file
Open in desktop

This file was deleted.

81 changes: 0 additions & 81 deletions Makefile
View file
Open in desktop

This file was deleted.

12 changes: 6 additions & 6 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,32 +6,32 @@

## Project setup
```
npm install
pnpm install
```

### Compiles and hot-reloads for development
```
npm run serve
pnpm run serve
Copy link
Collaborator

Choose a reason for hiding this comment

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

You need to let people know about this change.

And also let them know they need to do npm i -g pnpm before they can build this project.

```

### Compiles and minifies for production
```
npm run build
pnpm run build
```

### Run your tests
```
npm run test
pnpm run test
```

### Lints and fixes files
```
npm run lint
pnpm run lint
```

### Run your unit tests
```
npm run test:unit
pnpm run test:unit
```

### Customize configuration
Expand Down
Loading
Loading