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

Improve CI build speed #1956

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Improve CI build speed #1956

wants to merge 3 commits into from

Conversation

msdousti
Copy link
Collaborator

@msdousti msdousti commented Nov 10, 2024

Improve CI build speed by not running Maven package for the whole project. Using the past execution statistics, the build used to take 5 mins, which is now improved to 3 mins 🙂

Description

Due to the Maven shade plugin for logbook-servlet module, we need to run package once without running tests, and then run verify.
This can be improved by limiting the package step to the logbook-servlet module only (plus its dependencies)

Motivation and Context

This is a minor change to the CI build step, and improves the build speed.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

@msdousti
Copy link
Collaborator Author

@kasmarian As per discussion, added a build.sh file to streamline the build process, especially for local builds.

build.sh Show resolved Hide resolved
@msdousti
Copy link
Collaborator Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Minor changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants