You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@avro.apache.org by ie...@apache.org on 2021/06/11 14:59:56 UTC

[avro] 01/02: Address Rust PR review comments

This is an automated email from the ASF dual-hosted git repository.

iemejia pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/avro.git

commit b3944584e0dfea7c945902bde1d850b346aed4fd
Author: Ismaël Mejía <ie...@gmail.com>
AuthorDate: Fri Jun 11 14:09:50 2021 +0200

    Address Rust PR review comments
---
 .github/workflows/test-lang-rust-audit.yml  |  4 ++--
 .github/workflows/test-lang-rust-ci.yml     | 13 ++++++-------
 .github/workflows/test-lang-rust-clippy.yml |  3 +--
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/test-lang-rust-audit.yml b/.github/workflows/test-lang-rust-audit.yml
index 484f454..33b5113 100644
--- a/.github/workflows/test-lang-rust-audit.yml
+++ b/.github/workflows/test-lang-rust-audit.yml
@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
+name: Rust Security Audit
 on:
   workflow_dispatch:
   push:
@@ -26,14 +27,13 @@ on:
       - lang/rust/Cargo.toml
       - lang/rust/Cargo.lock
 
-name: Security Audit
 jobs:
   audit:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v2
-      - name: Audit
+      - name: Rust Audit
         uses: actions-rs/audit-check@v1
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.github/workflows/test-lang-rust-ci.yml b/.github/workflows/test-lang-rust-ci.yml
index b4086f1..449f558 100644
--- a/.github/workflows/test-lang-rust-ci.yml
+++ b/.github/workflows/test-lang-rust-ci.yml
@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
+name: Rust Continuous Integration
 on:
   workflow_dispatch:
   push:
@@ -25,8 +26,6 @@ on:
       - .github/workflows/test-lang-rust-ci.yml
       - lang/rust/**
 
-name: Continuous Integration
-
 jobs:
   ci:
     runs-on: ubuntu-latest
@@ -42,7 +41,7 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Toolchain
+      - name: Rust Toolchain
         uses: actions-rs/toolchain@v1
         with:
           profile: minimal
@@ -50,26 +49,26 @@ jobs:
           override: true
           components: rustfmt
 
-      - name: Format
+      - name: Rust Format
         uses: actions-rs/cargo@v1
         with:
           command: fmt
           args: --all -- --check
 
-      - name: Build
+      - name: Rust Build
         uses: actions-rs/cargo@v1
         with:
           command: build
           args: --all-features --all-targets
 
-      - name: Test
+      - name: Rust Test
         uses: actions-rs/cargo@v1
         with:
           command: test
           args: --all-features --all-targets
 
       # because of https://github.com/rust-lang/cargo/issues/6669
-      - name: Test docs
+      - name: Rust Test docs
         uses: actions-rs/cargo@v1
         with:
           command: test
diff --git a/.github/workflows/test-lang-rust-clippy.yml b/.github/workflows/test-lang-rust-clippy.yml
index d1f2d5c..9ac107b 100644
--- a/.github/workflows/test-lang-rust-clippy.yml
+++ b/.github/workflows/test-lang-rust-clippy.yml
@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
+name: Rust Clippy Check
 on:
   workflow_dispatch:
   push:
@@ -25,8 +26,6 @@ on:
       - .github/workflows/test-lang-rust-clippy.yml
       - lang/rust/**
 
-name: Clippy Check
-
 jobs:
   clippy_check:
     runs-on: ubuntu-latest