Skip to content

Commit

Permalink
Fix cli
Browse files Browse the repository at this point in the history
  • Loading branch information
@nebneb0703
nebneb0703 committed Sep 11, 2022
1 parent 3f75a4a commit e4ee882
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .dockerignore
View file
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/target/
**/*.rs.bk
.git
node-service-web/frontend
crates/node-service-web/frontend
16 changes: 8 additions & 8 deletions .github/workflows/rust.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ jobs:
node-version: '16.x'
- uses: actions/checkout@v3
- name: Install npm dependencies
run: cd node-service-web/frontend && npm install
run: cd crates/node-service-web/frontend && npm install
- name: Build the frontend
run: cd node-service-web/frontend && npm run build
run: cd crates/node-service-web/frontend && npm run build
- name: Build release
run: cargo build --release --verbose
- name: Run tests
Expand Down Expand Up @@ -60,9 +60,9 @@ jobs:
node-version: '16.x'
- uses: actions/checkout@v3
- name: Install npm dependencies
run: cd node-service-web/frontend && npm install
run: cd crates/node-service-web/frontend && npm install
- name: Build the frontend
run: cd node-service-web/frontend && npm run build
run: cd crates/node-service-web/frontend && npm run build
- name: Build release
run: cargo build --release --verbose --workspace --exclude pluto-coordinator
- name: Run tests
Expand All @@ -83,9 +83,9 @@ jobs:
node-version: '16.x'
- uses: actions/checkout@v3
- name: Install npm dependencies
run: cd node-service-web/frontend && npm install
run: cd crates/node-service-web/frontend && npm install
- name: Build the frontend
run: cd node-service-web/frontend && npm run build
run: cd crates/node-service-web/frontend && npm run build
- name: Build release
run: cargo build --release --verbose --workspace --exclude pluto-coordinator
- name: Run tests
Expand All @@ -102,9 +102,9 @@ jobs:
steps:
- uses: actions/checkout@v3
- name: Install npm dependencies
run: cd node-service-web/frontend && npm install

This comment has been minimized.

Sign in to view

Copy link
@nebneb0703

nebneb0703 Sep 11, 2022

Author Collaborator

@profi248 do we even need to test the frontend with miri? we can probably remove npm from this task and exclude that package

This comment has been minimized.

Sign in to view

Copy link
@profi248

profi248 Sep 11, 2022

Owner

yeah, it would be best to just exclude that crate
run: cd crates/node-service-web/frontend && npm install
- name: Build the frontend
run: cd node-service-web/frontend && npm run build
run: cd crates/node-service-web/frontend && npm run build
- name: Install Miri
run: |
rustup toolchain install nightly --component miri
Expand Down

1 comment on commit e4ee882

@nebneb0703
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ci not cli lol oops

Please sign in to comment.