Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
Menu
Open sidebar
OpenDAS
dynamo
Commits
ffbc06cc
Commit
ffbc06cc
authored
Feb 05, 2025
by
Anant Sharma
Committed by
GitHub
Feb 05, 2025
Browse files
ci: update rust pre-merge checks (#97)
parent
ccfe101e
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
60 additions
and
35 deletions
+60
-35
.github/workflows/pre-merge-rust.yml
.github/workflows/pre-merge-rust.yml
+60
-0
.github/workflows/pre-merge.yml
.github/workflows/pre-merge.yml
+0
-35
No files found.
.github/workflows/pre-merge-rust.yml
0 → 100644
View file @
ffbc06cc
# SPDX-FileCopyrightText: Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
name
:
Rust pre-merge checks
on
:
# Always run this workflow when commits are pushed to main.
push
:
branches
:
-
main
# Run this workflow on pull requests targeting main but only if files in runtime/rust change.
pull_request
:
branches
:
-
main
paths
:
-
'
runtime/rust/**'
jobs
:
pre-merge-rust
:
runs-on
:
ubuntu-latest
permissions
:
contents
:
read
steps
:
-
uses
:
actions/checkout@v4
-
name
:
Set up system dependencies
run
:
|
sudo apt-get update -y
sudo apt-get install -y protobuf-compiler
-
name
:
Cache Cargo Registry
uses
:
actions/cache@v3
with
:
path
:
~/.cargo/registry
key
:
${{ runner.os }}-cargo-registry-${{ github.head_ref || github.ref_name }}-${{ hashFiles('**/Cargo.lock') }}
-
name
:
Set up Rust Toolchain Components
run
:
rustup component add rustfmt clippy
-
name
:
Run Cargo Check
working-directory
:
runtime/rust
run
:
cargo check --locked
-
name
:
Verify Code Formatting
working-directory
:
runtime/rust
run
:
cargo fmt -- --check
-
name
:
Run Clippy Checks
working-directory
:
runtime/rust
run
:
cargo clippy --no-deps --all-targets -- -D warnings
-
name
:
Run Unit Tests
working-directory
:
runtime/rust
run
:
cargo test --locked --all-targets
\ No newline at end of file
.github/workflows/pre-merge.yml
View file @
ffbc06cc
...
...
@@ -88,38 +88,3 @@ jobs:
# - run: ./icp/protos/gen_python.sh
# - run: pytest -p no:warnings --verbose worker/python/tests
# timeout-minutes: 2
pre-merge-rust
:
runs-on
:
ubuntu-latest
permissions
:
contents
:
read
steps
:
-
uses
:
actions/checkout@v4
-
name
:
Set up system dependencies
run
:
|
sudo apt-get update -y
sudo apt-get install -y protobuf-compiler
-
name
:
Cache Cargo Registry
uses
:
actions/cache@v3
with
:
path
:
~/.cargo/registry
key
:
${{ runner.os }}-cargo-registry-${{ hashFiles('**/Cargo.lock') }}
-
name
:
Cache Cargo Target Directory
uses
:
actions/cache@v3
with
:
path
:
runtime/rust/target
key
:
${{ runner.os }}-cargo-target-${{ hashFiles('**/Cargo.lock') }}
-
name
:
Set up Rust Toolchain Components
run
:
rustup component add rustfmt clippy
-
name
:
Run Cargo Check
working-directory
:
runtime/rust
run
:
cargo check --locked
-
name
:
Verify Code Formatting
working-directory
:
runtime/rust
run
:
cargo fmt -- --check
-
name
:
Run Clippy Checks
working-directory
:
runtime/rust
run
:
cargo clippy --no-deps --all-targets -- -D warnings
-
name
:
Run Unit Tests
working-directory
:
runtime/rust
run
:
cargo test --locked --all-targets
\ No newline at end of file
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment