Skip to content

Commit

Permalink
Merge pull request #31 from jakedeichert/ci-pullrequest
Browse files Browse the repository at this point in the history
Try to make GitHub Actions run CI tests for pull requests too
  • Loading branch information
jacobdeichert authored Sep 24, 2019
2 parents b704f63 + 5c30610 commit 1884caa
Showing 1 changed file with 24 additions and 25 deletions.
49 changes: 24 additions & 25 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,41 +1,40 @@
name: CI

on: push
on: [push, pull_request]

jobs:

build:
name: Build
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
- name: Fetch dependencies
run: cargo fetch
- name: Build in release mode
run: cargo build --release --frozen
- uses: actions/checkout@master
- name: Fetch dependencies
run: cargo fetch
- name: Build in release mode
run: cargo build --release --frozen

test:
name: Test
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
- name: Add Ruby for a test that requires it
uses: actions/setup-ruby@master
with:
version: 2.6.x
- name: Fetch dependencies
run: cargo fetch
- name: Build in test mode
run: cargo build --tests --frozen
- name: Make mask available globally for certain test suites that depend on it
run: cp ./target/debug/mask /usr/share/rust/.cargo/bin
- name: Run tests
run: cargo test --frozen
- uses: actions/checkout@master
- name: Add Ruby for a test that requires it
uses: actions/setup-ruby@master
with:
version: 2.6.x
- name: Fetch dependencies
run: cargo fetch
- name: Build in test mode
run: cargo build --tests --frozen
- name: Make mask available globally for certain test suites that depend on it
run: cp ./target/debug/mask /usr/share/rust/.cargo/bin
- name: Run tests
run: cargo test --frozen

format:
name: Format
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
- name: Verify formatting is correct
run: cargo fmt --all -- --check
- uses: actions/checkout@master
- name: Verify formatting is correct
run: cargo fmt --all -- --check

0 comments on commit 1884caa

Please sign in to comment.