Skip to content

Commit

Permalink
Add scalafmtCheckAll to pr build (bors/branch build to come later)
Browse files Browse the repository at this point in the history
This will fail the PR build if code isn't formatted properly.
This won't fail builds done by bors, so that it fulfills only
informational role for now. We might decide to enforce that for in bors
builds later, depending on the results.

Parallel execution is disabled for scalafmtCheckAll in CI.
See scalameta/scalafmt#1399 for reasons why.
  • Loading branch information
ArturGajowy committed Apr 16, 2019
1 parent 33060bf commit 346668a
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions .drone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,15 @@ pipeline:

# pr build

pr-lint:
<<: *pr_build_conditions
group: parallel
<<: *sbttestenv
commands:
- cp -r . ../step.lint && cd ../step.lint
# Disable parallelism during scalafmtCheckAll. See https://github.com/scalameta/scalafmt/issues/1399.
- sbt --info "set Global / concurrentRestrictions := Seq(Tags.limitAll(1))" scalafmtCheckAll

pr-run-unit-tests:
<<: *pr_build_conditions
group: parallel
Expand Down

0 comments on commit 346668a

Please sign in to comment.