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

Implementing GitHub Actions Pipeline #982

Draft
wants to merge 121 commits into
base: dev
Choose a base branch
from

Conversation

PhilippSchmelter
Copy link
Contributor

@PhilippSchmelter PhilippSchmelter commented Oct 2, 2024

Resolves #939

@PhilippSchmelter PhilippSchmelter added the ci Related to continuous integration label Oct 2, 2024
@PhilippSchmelter PhilippSchmelter self-assigned this Oct 2, 2024
Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

I know this is very much still work in progress, but here's some small things I noticed while looking over ci.yml

Comment on lines +13 to +15
- 'feature/*'
- 'hotfix/*'
- 'release/*'
Copy link
Member

Choose a reason for hiding this comment

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

You probably know this, but these have to be adapted to the branch naming scheme that we use

Comment on lines +18 to +19
- main
- dev
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need the pull_request specification, if CI is running on branches already?

name: CI

on:
push:
Copy link
Member

Choose a reason for hiding this comment

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

paths-ignore for docs could also be useful? readthedocs has its separate CI

Comment on lines +48 to +49
with:
gradle-version: "8.10"
Copy link
Member

Choose a reason for hiding this comment

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

Seems like you can leave out the specification of the version if the project already configures gradle (which it does).


- name: Run Tests
run: |
./gradlew --refresh-dependencies spotlessCheck pmdMain pmdTest
Copy link
Member

Choose a reason for hiding this comment

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

--refresh-dependencies makes more sense before assemble, I think

id: sonarqube-quality-gate-check
uses: sonarsource/sonarqube-quality-gate-action@master
# Force to fail step after specific time.
timeout-minutes: 60
Copy link
Member

Choose a reason for hiding this comment

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

This could probably be reduced (but isn't that important)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Related to continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Evaluate whether move to Github Actions makes sense
2 participants