Skip to content

Conversation

@stempler
Copy link
Member

@stempler stempler commented May 2, 2025

BREAKING CHANGE: Behavior of skip-build parameter for gradle-library.yml and gradle-service.yml workflows changed.

BREAKING CHANGE: Behavior of skip-build parameter for gradle-library.yml
and gradle-service.yml workflows changed.
@stempler stempler requested a review from Copilot May 2, 2025 09:24
Copy link

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 modifies the CI workflows for Gradle projects by changing the behavior of the skip-build parameter and introducing a new parameter for publish workflows.

  • Modified skip-build description and related conditional checks in gradle-service.yml and gradle-library.yml
  • Introduced a skip-check parameter in gradle-service-publish.yml and gradle-library-publish.yml, mapped to skip-build in job inputs

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/gradle-service.yml Updated skip-build description and removed its condition from if-clauses
.github/workflows/gradle-service-publish.yml Added skip-check parameter and mapped it to skip-build
.github/workflows/gradle-library.yml Updated skip-build description and adjusted conditional logic
.github/workflows/gradle-library-publish.yml Added skip-check parameter and mapped it to skip-build
Comments suppressed due to low confidence (2)

.github/workflows/gradle-service-publish.yml:38

  • [nitpick] The parameter 'skip-check' is introduced in this workflow while other configurations use 'skip-build'. Consider renaming it to 'skip-build' for consistency across workflows.
skip-check:

.github/workflows/gradle-library-publish.yml:26

  • [nitpick] The new 'skip-check' parameter here could be renamed to 'skip-build' to align with similar settings in other workflows, improving consistency.
skip-check:

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