Skip to content

Conversation

@tschut
Copy link
Contributor

@tschut tschut commented May 19, 2025

This fixes #399

(cherry picked from commit b7c0dc1)
@tschut tschut marked this pull request as ready for review May 19, 2025 14:13
@tschut tschut requested a review from a team as a code owner May 19, 2025 14:13
@psx95
Copy link
Contributor

psx95 commented May 22, 2025

/gcbrun

Copy link
Contributor

@psx95 psx95 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

It seems like you need to run ./gradlew spotlessApply to fix some style checks.

@tschut
Copy link
Contributor Author

tschut commented May 23, 2025

Thanks @psx95, styling and imports should now be ok!

@psx95
Copy link
Contributor

psx95 commented May 24, 2025

/gcbrun

@psx95
Copy link
Contributor

psx95 commented May 24, 2025

Hi @tschut, the CLA check is failing in the recent commits. Could you fix that ?

I think it might be because you added commits that were created using a different email account not covered by the CLA. 

@tschut
Copy link
Contributor Author

tschut commented May 26, 2025

Hi @tschut, the CLA check is failing in the recent commits. Could you fix that ?

I think it might be because you added commits that were created using a different email account not covered by the CLA.

Crap. Rewrote a bit of history and I think it's correct now.

@tschut tschut requested a review from psx95 May 27, 2025 09:09
@psx95
Copy link
Contributor

psx95 commented May 27, 2025

/gcbrun

Copy link
Contributor

@psx95 psx95 left a comment

Choose a reason for hiding this comment

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

Thanks for this PR!

@psx95 psx95 merged commit e546d0b into GoogleCloudPlatform:main May 27, 2025
18 checks 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.

Update usage of removed ResourceAttributes

2 participants