Skip to content

Fixup our CI#988

Merged
nedtwigg merged 14 commits intomainfrom
feat/fix-ci
Nov 9, 2021
Merged

Fixup our CI#988
nedtwigg merged 14 commits intomainfrom
feat/fix-ci

Conversation

@nedtwigg
Copy link
Copy Markdown
Member

@nedtwigg nedtwigg commented Nov 9, 2021

We had various CI rot issues as documented in #930, all fixed here. In particular:

  • we now run tests against Java 8 again
  • we run spotbugs again
  • we retry test failures on CI
  • we store npm test results on Circle, as well as spotbugs results

Also fixed-up some spotbugs findings - a few unused fields and unused temp variables.

Base automatically changed from feat/spotless-task-service to main November 9, 2021 05:59
@nedtwigg nedtwigg merged commit 7cc87da into main Nov 9, 2021
@nedtwigg nedtwigg deleted the feat/fix-ci branch November 9, 2021 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant