Skip to content

add a smoke-test mvn profile - #3085

Merged
neozwu merged 2 commits into
googleapis:masterfrom
neozwu:smoketest-profile
Mar 28, 2018
Merged

add a smoke-test mvn profile#3085
neozwu merged 2 commits into
googleapis:masterfrom
neozwu:smoketest-profile

Conversation

@neozwu

Copy link
Copy Markdown
Contributor

This change adds a mvn profile to run smoke tests (by invoking "mvn -P smoke-test verify") once a separate change (googleapis/gapic-generator#1898) is used to generate code.

@neozwu
neozwu requested a review from vam-googleMarch 26, 2018 21:26
@neozwu
neozwu requested a review from pongad as a code ownerMarch 26, 2018 21:26
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 26, 2018
Comment threadpom.xml Outdated
</includes>
<excludes>
<exclude>**/*IT.java</exclude>
</excludes>

This comment was marked as spam.

@neozwu

Copy link
Copy Markdown
ContributorAuthor

@garrettjonesgoogle PTAL

@garrettjonesgooglegarrettjonesgoogle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@vam-googlevam-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@neozwu
neozwu merged commit 89082b3 into googleapis:masterMar 28, 2018
suztomo pushed a commit that referenced this pull request Mar 9, 2026
* chore: Create renovate_config_check.yaml
Source-Link: googleapis/synthtool@6612ab8
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:a6aa751984f1e905c3ae5a3aac78fc7b68210626ce91487dc7ff4f0a06f010cc
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
…ase-please (#3091)
fixes#3085
This change should migrate all required tests from "settings" tab to
`.github/sync-repo-settings.yaml`. Also setting up release-please
branches for lts.
Sync-repo-settings.yaml is updated referencing [this
guide](https://github.com/googleapis/repo-automation-bots/tree/main/packages/sync-repo-settings),
and settings in google-cloud-java
[here](https://github.com/googleapis/google-cloud-java/blob/main/.github/sync-repo-settings.yaml).
Release-please.yaml is updated referencing
[this](https://github.com/googleapis/repo-automation-bots/tree/main/packages/release-please#configuration)
and google-cloud-java settings
[here](https://github.com/googleapis/google-cloud-java/blob/main/.github/release-please.yml).
**Questions to reviewers:**
- Is permission rules with "yoshi-" teams still accurate? Do we prefer
java specific teams as admin/push?
- This update will override configurations in “Settings” for protected
branches, I do not have access to this tab. Can you please double check
I am not missing anything AND/OR give me access to it?
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
Uses the gson Java version checker, which is more production-hardened than the custom one in the Spanner client. It seems that the Spanner implementation does not get it right on all versions on Windows.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@neozwu@garrettjonesgoogle@vam-google@googlebot