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

Rework CI workflow names #306

Merged
merged 1 commit into from
Jun 6, 2024
Merged

Rework CI workflow names #306

merged 1 commit into from
Jun 6, 2024

Conversation

sgraband
Copy link
Contributor

@sgraband sgraband commented Jun 6, 2024

Prefix each name with the type (MAVEN, DOCKER, NPM).
Use short names for each component.
Remove the long name for workflow calls, simply call them run.
Merge jobs together to narrow down the amount of jobs being called.

Screenshot:
image

Prefix each name with the type (MAVEN, DOCKER, NPM).
Use short names for each component.
Remove the long name for workflow calls, simply call them `run`.
Merge jobs together to narrow down the amount of jobs being called.
@sgraband sgraband requested a review from jfaltermeier June 6, 2024 12:24
@sgraband sgraband marked this pull request as ready for review June 6, 2024 12:24
@lucas-koehler lucas-koehler requested review from lucas-koehler and removed request for jfaltermeier June 6, 2024 14:24
Copy link
Contributor

@lucas-koehler lucas-koehler left a comment

Choose a reason for hiding this comment

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

LGTM thanks! The namings are much cleaner now ✨

@sgraband sgraband merged commit 3e50954 into main Jun 6, 2024
1 check passed
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