devirgo style on phase 1 #23

	name: Lints

	# We only run these lints on trial-merges of PRs to reduce noise.
	on:
	merge_group:
	pull_request:
	types: [synchronize, opened, reopened, ready_for_review]
	push:
	branches:
	- master

	jobs:
	skip_check:
	runs-on: ubuntu-latest
	outputs:
	should_skip: ${{ steps.skip_check.outputs.should_skip }}
	steps:
	- id: skip_check
	uses: fkirc/skip-duplicate-actions@v5
	with:
	cancel_others: 'true'
	concurrent_skipping: 'same_content_newer'
	paths_ignore: '["**/README.md"]'

	lints:
	needs: [skip_check]
	if: \|
	github.event.pull_request.draft == false &&
	(github.event.action == 'ready_for_review' \|\| needs.skip_check.outputs.should_skip != 'true')

	name: Various lints
	timeout-minutes: 30
	runs-on: ubuntu-latest

	steps:
	- uses: actions/checkout@v2
	- uses: actions-rs/toolchain@v1
	with:
	components: rustfmt, clippy
	override: false
	- name: Cargo cache
	uses: actions/cache@v3
	with:
	path: \|
	~/.cargo/bin/
	~/.cargo/registry/index/
	~/.cargo/registry/cache/
	~/.cargo/git/db/
	target/
	key: lint-${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}

	- name: Install cargo make
	run: \|
	cargo install --force cargo-make
	- name: Check code format
	uses: actions-rs/cargo@v1
	with:
	command: make
	args: fmt-check

	- name: Run clippy
	uses: actions-rs/cargo@v1
	with:
	command: make
	args: clippy

Provide feedback