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

chore: make the showcase scripts to automatically prepare the generator jar #3171

Merged
merged 4 commits into from
Sep 10, 2024

Conversation

diegomarquezp
Copy link
Contributor

The recent changes in the hermetic library generation workflow make it necessary for the environment to have a well-known folder where the generator jar is assumed to be located.

This PR sets up the generator jar in such location when running showcase golden tests or using the -Pupdate profile.

More details in the development guide

@product-auto-label product-auto-label bot added the size: s Pull request size is small. label Sep 5, 2024
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Sep 5, 2024
@@ -76,7 +76,11 @@ Note:

## Update the Golden Showcase Files

Open a new terminal window in the root project directory.
Open a new terminal window in the root project directory. Note that the
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: one sentence per line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, done.

@diegomarquezp diegomarquezp added automerge Merge the pull request once unit tests and other checks pass. automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit and removed automerge Merge the pull request once unit tests and other checks pass. labels Sep 10, 2024
Copy link

sonarcloud bot commented Sep 10, 2024

Copy link

sonarcloud bot commented Sep 10, 2024

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@gcf-merge-on-green gcf-merge-on-green bot merged commit badca93 into main Sep 10, 2024
47 of 49 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Sep 10, 2024
@gcf-merge-on-green gcf-merge-on-green bot deleted the fix-showcase-generator-well-known-location branch September 10, 2024 18:10
lqiu96 pushed a commit that referenced this pull request Sep 11, 2024
…or jar (#3171)

The recent changes in the hermetic library generation workflow make it necessary for the environment to have a well-known folder where the generator jar is assumed to be located.

This PR sets up the generator jar in such location when running showcase golden tests or using the `-Pupdate` profile.

More details in the [development guide](https://github.com/googleapis/sdk-platform-java/blob/main/library_generation/DEVELOPMENT.md#the-hermetic-builds-well-known-folder)
ldetmer pushed a commit that referenced this pull request Sep 17, 2024
…or jar (#3171)

The recent changes in the hermetic library generation workflow make it necessary for the environment to have a well-known folder where the generator jar is assumed to be located.

This PR sets up the generator jar in such location when running showcase golden tests or using the `-Pupdate` profile.

More details in the [development guide](https://github.com/googleapis/sdk-platform-java/blob/main/library_generation/DEVELOPMENT.md#the-hermetic-builds-well-known-folder)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants