Skip to content

Conversation

@0marperez
Copy link
Contributor

Issue #

Description of changes

Gets rid of changes cause by configure Gradle workflow step

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@0marperez 0marperez added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Feb 14, 2025
@github-actions
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

@0marperez 0marperez marked this pull request as ready for review February 14, 2025 19:49
@0marperez 0marperez requested a review from a team as a code owner February 14, 2025 19:49
@github-actions
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

@github-actions

This comment has been minimized.

run: |
echo "merging main @ $main_sha into release @ $release_sha";
# Getting rid of the changes caused by "Configure Gradle"
Copy link
Member

Choose a reason for hiding this comment

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

This comment could be more specific. What changes are being undone, why?

Copy link
Contributor

@ianbotsf ianbotsf left a comment

Choose a reason for hiding this comment

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

Approved pending small change.

Comment on lines 94 to 97
# Getting rid of the changes caused by "Configure Gradle"
git stash
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: As noted in offline conversation, git stash -- gradle/wrapper/gradle-wrapper.properties would be a safer command here.

@sonarqubecloud
Copy link

@github-actions
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK

@github-actions
Copy link

Affected Artifacts

No artifacts changed size

@0marperez 0marperez changed the title fix: update release branch workflow having Gradle URL code changes fix: update-release-branch workflow having Gradle URL code changes Feb 14, 2025
@0marperez 0marperez merged commit 3949900 into main Feb 14, 2025
19 checks passed
@0marperez 0marperez deleted the fix-release-branch-wf branch February 14, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants