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.

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

Minor refresh #46

Merged
merged 7 commits into from
Jul 3, 2021
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
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
66 changes: 66 additions & 0 deletions .github/workflows/quick-check.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,66 @@
# This workflow runs on every push and checks whether everything looks good

name: Quick check

on:
push:
tags-ignore:
- v*
pull_request:
types: [opened, synchronize, reopened]

workflow_dispatch:

jobs:
quick_check-format:
runs-on: ubuntu-latest
steps:
- name: Install Rust stable toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
components: rustfmt

- uses: actions/checkout@v2

- name: Cargo fmt
uses: actions-rs/cargo@v1
with:
command: fmt
args: --all -- --check

quick_check-clippy:
runs-on: ubuntu-latest
steps:
- name: Install Rust stable toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
components: clippy

- uses: actions/checkout@v2

- name: Cargo clippy
uses: actions-rs/cargo@v1
with:
command: clippy
args: -- -D warnings

quick_check-tests:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2

- name: Cargo test
uses: actions-rs/cargo@v1
with:
command: test

- name: Cargo check
uses: actions-rs/cargo@v1
with:
command: check
Copy link
Owner

Choose a reason for hiding this comment

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

why add that compared to the current pipeline?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did not notice your pipeline at first :)
I also cannot run your pipeline in my fork (beside when opening a PR) so the github workflow makes it easier to check on my end.

I can remove it from the PR if you prefer.

Copy link
Owner

Choose a reason for hiding this comment

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

yes please, no need for 2 CI setup

Loading