Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI pipeline improvements #424

Merged
merged 1 commit into from
Jun 18, 2024
Merged

CI pipeline improvements #424

merged 1 commit into from
Jun 18, 2024

Conversation

nscuro
Copy link
Member

@nscuro nscuro commented Jun 15, 2024

  • Only build against Temurin, drop Zulu
  • Build against Java 8, 11, 17, and 21, instead of just 8
  • Pin 3rd party actions to their digest
  • Drop permissions on workflow-level, define them on job-level
  • Update JamesIves/github-pages-deploy-action to the latest version
  • Update release-drafter/release-drafter to the latest version
  • Run the Maven CI workflow only for pushes to, or pull request into, master. Previously we had duplicate runs for PRs.
  • Reduce build output noise by disabling Maven's transfer progress logs
  • Record test coverage with JaCoCo
  • Publish coverage reports to Codacy
    • Because we build against multiple Java versions, we'll upload only the report generated from the latest Java build

* Only build against Temurin, drop Zulu
* Build against Java 8, 11, 17, and 21, instead of just 8
* Pin 3rd party actions to their digest
* Drop `permissions` on workflow-level, define them on job-level
* Update `JamesIves/github-pages-deploy-action` to the latest version
* Update `release-drafter/release-drafter` to the latest version
* Run the `Maven CI` workflow only for pushes to, or pull request into, `master`. Previously we had duplicate runs for PRs.
* Reduce build output noise by disabling Maven's transfer progress logs
* Record test coverage with JaCoCo
* Publish coverage reports to Codacy
  * Because we build against multiple Java versions, we'll upload only the report generated from the latest Java build

Signed-off-by: nscuro <nscuro@protonmail.com>
Copy link
Contributor

@mr-zepol mr-zepol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 LGTM

@nscuro nscuro merged commit a9ca98e into master Jun 18, 2024
9 checks passed
@nscuro nscuro deleted the improve-ci-pipeline branch June 18, 2024 13:43
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.

2 participants