Skip to content
This repository was archived by the owner on Aug 12, 2025. It is now read-only.

Conversation

@mzcydev
Copy link
Member

@mzcydev mzcydev commented Jul 10, 2025

No description provided.

@mzcydev mzcydev requested a review from Copilot July 10, 2025 17:48

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mzcydev mzcydev requested a review from Copilot July 10, 2025 17:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the release workflow to include the dev branch and adjust prerelease logic.

  • Triggers the release workflow on pushes to dev in addition to master/main
  • Converts tag-exists messages from German to English
  • Modifies prerelease flag to always prerelease on dev or when version ends with -SNAPSHOT
Comments suppressed due to low confidence (2)

.github/workflows/release.yml:8

  • Consider adding or updating tests (or documentation) to verify that pushes to the new dev branch correctly trigger the workflow and set prerelease: true.
    branches: [ master, main, dev ]   # ← Added "dev" here

.github/workflows/release.yml:85

  • [nitpick] Extract the hardcoded branch ref refs/heads/dev into an environment variable or workflow input to avoid magic strings and improve maintainability.
          prerelease: ${{ (github.ref == 'refs/heads/dev') || (endsWith(steps.vars.outputs.version, '-SNAPSHOT')) }}

@mzcydev mzcydev merged commit 5072379 into master Jul 10, 2025
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants