Transit CI to GitHub Actions #1942
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,216 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - auto | ||
| - try | ||
|
|
||
| pull_request: | ||
|
|
||
| jobs: | ||
| frontend: | ||
| name: Frontend | ||
| runs-on: ubuntu-16.04 | ||
| env: | ||
| JOBS: 1 # See https://git.io/vdao3 for details. | ||
| PERCY_PARALLEL_TOTAL: 2 | ||
| # Percy secrets are included here to enable Percy's GitHub integration | ||
| # on community-submitted PRs | ||
| PERCY_TOKEN: 0d8707a02b19aebbec79bb0bf302b8d2fa95edb33169cfe41b084289596670b1 | ||
| PERCY_PROJECT: crates-io/crates.io | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v2-beta | ||
|
|
||
| - uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.npm | ||
| key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-node- | ||
|
|
||
| - uses: actions/setup-node@v1 | ||
| with: | ||
| node-version: '12.x' | ||
|
|
||
| - name: Install node modules | ||
| run: npm ci | ||
|
|
||
| - name: Lint | ||
| run: | | ||
| npm run lint:hbs | ||
| npm run lint:js | ||
| npm run lint:deps | ||
|
|
||
| - name: Add nonce for parallel tests | ||
| run: echo ::set-env name=PERCY_PARALLEL_NONCE::`date +%s` | ||
|
|
||
| - name: Test | ||
| run: npm test | ||
|
|
||
| backend: | ||
| name: Backend | ||
| runs-on: ubuntu-16.04 | ||
| strategy: | ||
| # TODO: [ci] should be true if GitHub Actions supports ""allow failures" on matrix | ||
| fail-fast: false | ||
| matrix: | ||
| rust: | ||
| - stable | ||
| - beta | ||
| - nightly | ||
|
|
||
| env: | ||
| DATABASE_URL: postgres://postgres:@localhost/cargo_registry_test | ||
| TEST_DATABASE_URL: postgres://postgres:@localhost/cargo_registry_test | ||
| CARGO_INCREMENTAL: 0 | ||
| RUSTFLAGS: "-C debuginfo=0 -D warnings" | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v2-beta | ||
|
|
||
| - name: Cleanup pre-installed Rust toolchains | ||
| # The pre-installed toolchain is under root permission, and this would | ||
| # make tarball fail to extract. Here we remove the symlink and install | ||
| # our own Rust toolchain if necessary. | ||
| run: | | ||
| which rustup | ||
| which cargo | ||
| if [[ -L "$HOME/.cargo" && -L "$HOME/.rustup" ]]; then | ||
| rm -v "$HOME/.rustup" | ||
| rm -v "$HOME/.cargo" | ||
| fi | ||
| echo "::add-path::$HOME/.cargo/bin" | ||
|
|
||
| - name: Cache rustup | ||
| uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.rustup | ||
| key: ${{ runner.os }}-rustup-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
|
There was a problem hiding this comment. I'm not sure if it is necessary to include the By the same reasoning, If you're unsure about removing it though, I definitely don't see this as a blocker. This is probably a micro-optimization given the expected size of these directories. There was a problem hiding this comment. Thanks. We need figure out when to invalidate old cache. Maybe some trick like script//ci/cargo-clean-on-new-rustc-version.sh, but I am not sure how to implement that. |
||
| restore-key: | | ||
| ${{ runner.os }}-rustup-${{ matrix.rust }}- | ||
| ${{ runner.os }}-rustup- | ||
|
|
||
| - name: Cache cargo binaries | ||
weihanglo marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.cargo/bin | ||
| key: ${{ runner.os }}-cargo-bin-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-key: | | ||
| ${{ runner.os }}-cargo-bin-${{ matrix.rust }}- | ||
| ${{ runner.os }}-cargo-bin- | ||
|
|
||
| - name: Cache cargo registry cache | ||
| uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.cargo/registry/cache | ||
| key: ${{ runner.os }}-cargo-registry-cache-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-key: | | ||
| ${{ runner.os }}-cargo-registry-cache-${{ matrix.rust }}- | ||
| ${{ runner.os }}-cargo-registry-cache- | ||
|
|
||
| - name: Cache cargo registry index | ||
| uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.cargo/registry/index | ||
| key: ${{ runner.os }}-cargo-registry-index-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-key: | | ||
| ${{ runner.os }}-cargo-registry-index-${{ matrix.rust }}- | ||
| ${{ runner.os }}-cargo-registry-index- | ||
|
|
||
| - name: Cache cargo git db | ||
| uses: actions/cache@v1 | ||
| with: | ||
| path: ~/.cargo/git/db | ||
| key: ${{ runner.os }}-cargo-git-db-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-key: | | ||
| ${{ runner.os }}-cargo-git-db-${{ matrix.rust }}- | ||
| ${{ runner.os }}-cargo-git-db- | ||
|
|
||
| - name: Cache cargo build | ||
| uses: actions/cache@v1 | ||
| with: | ||
| path: target | ||
| key: ${{ runner.os }}-cargo-build-target-${{ matrix.rust }}-${{ hashFiles('**/Cargo.lock') }} | ||
| restore-key: | | ||
| ${{ runner.os }}-cargo-build-target-${{ matrix.rust }}- | ||
| ${{ runner.os }}-cargo-build-target- | ||
|
|
||
| - name: Install ${{ matrix.rust }} Rust | ||
| run: | | ||
| if [[ ! -d "$HOME/.cargo" || ! -d "$HOME/.rustup" ]]; then | ||
| curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs > rustup-init.sh | ||
| sh rustup-init.sh -y --default-toolchain none | ||
| fi | ||
| rustup set profile minimal | ||
| rustup update ${{ matrix.rust }} | ||
| rustup default ${{ matrix.rust }} | ||
|
|
||
| - name: Install lint tools | ||
| if: matrix.rust == 'stable' | ||
| run: | | ||
| rustup component add rustfmt | ||
| rustup component add clippy | ||
|
|
||
| - name: Cargo clean on new rustc version | ||
| run: script/ci/cargo-clean-on-new-rustc-version.sh | ||
|
|
||
| - name: Install required system libraries | ||
| run: | | ||
| sudo tee /etc/apt/sources.list.d/pgdg.list <<END | ||
| deb http://apt.postgresql.org/pub/repos/apt/ $(lsb_release -cs)-pgdg main | ||
| END | ||
| wget --quiet -O - https://www.postgresql.org/media/keys/ACCC4CF8.asc | sudo apt-key add - | ||
| sudo apt-get update | ||
| sudo apt-get install -y libpq-dev postgresql-11 | ||
| mkdir -p /etc/postgresql/11/main | ||
| sudo tee /etc/postgresql/11/main/pg_hba.conf <<END | ||
| local all all trust | ||
| host all all 127.0.0.1/32 trust | ||
| END | ||
| sudo systemctl restart postgresql@11-main | ||
|
|
||
| - name: Setup database | ||
| run: | | ||
| which diesel || cargo install diesel_cli --vers $(cat .diesel_version) --no-default-features --features postgres | ||
| diesel database setup --locked-schema | ||
|
|
||
| - name: Lint | ||
| if: matrix.rust == 'stable' | ||
| run: | | ||
| cargo fmt -- --check | ||
| cargo clippy --all-targets --all-features --all | ||
|
|
||
| - name: Test | ||
| run: cargo test | ||
|
|
||
| - name: Prune unnecessary cache | ||
| run: script/ci/prune-cache.sh | ||
|
|
||
|
|
||
| # These jobs doesn't actually test anything, but they're only used to tell | ||
| # bors the build completed, as there is no practical way to detect when a | ||
| # workflow is successful listening to webhooks only. | ||
| # | ||
| # ALL THE PREVIOUS JOBS NEEDS TO BE ADDED TO THE `needs` SECTION OF THIS JOB! | ||
|
|
||
| end-success: | ||
| name: bors build finished | ||
| if: success() | ||
| runs-on: ubuntu-latest | ||
| needs: [frontend, backend] | ||
|
|
||
| steps: | ||
| - name: Mark the job as successful | ||
| run: exit 0 | ||
|
|
||
| end-failure: | ||
| name: bors build finished | ||
| if: "!success()" | ||
| runs-on: ubuntu-latest | ||
| needs: [frontend, backend] | ||
|
|
||
| steps: | ||
| - name: Mark the job as a failure | ||
| run: exit 1 | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On travis we need to include the
masterbranch, because it's (well technically the repoHEAD) cache is used as the fallback for new branches and PRs. That doesn't seem to apply to the caching logic on GH Actions, so I think its okay to drop it here.