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.

Sign up for GitHub

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

Jump to bottom

feat: introduce AdsMarketingPlatformAdmin #7697

Merged

bshaffer merged 2 commits into main from introduce-marketingplatformadmin

Sep 27, 2024

+3,263 −0

Contributor

bshaffer commented Sep 24, 2024

b/348678923


          feat: introduce AdsMarketingPlatformAdmin

e0240cf

bshaffer requested review from a team as code owners

September 24, 2024 20:10

snippet-bot bot commented Sep 24, 2024 •

edited

Loading

Here is the summary of possible violations 😱

There are 5 possible violations for not having product prefix.

AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/create_analytics_account_link.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_CreateAnalyticsAccountLink_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/delete_analytics_account_link.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_DeleteAnalyticsAccountLink_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/get_organization.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_GetOrganization_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/list_analytics_account_links.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_ListAnalyticsAccountLinks_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/set_property_service_level.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_SetPropertyServiceLevel_sync

The end of the violation section. All the stuff below is FYI purposes only.

Here is the summary of changes.

You are about to add 5 region tags.

AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/create_analytics_account_link.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_CreateAnalyticsAccountLink_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/delete_analytics_account_link.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_DeleteAnalyticsAccountLink_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/get_organization.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_GetOrganization_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/list_analytics_account_links.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_ListAnalyticsAccountLinks_sync
AdsMarketingPlatformAdmin/samples/V1alpha/MarketingplatformAdminServiceClient/set_property_service_level.php:25, tag marketingplatformadmin_v1alpha_generated_MarketingplatformAdminService_SetPropertyServiceLevel_sync

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

Refresh this comment

bshaffer added the next release label

Hectorhammett approved these changes

View reviewed changes


          typo fix

463a1bb

bshaffer merged commit abe3c50 into main

23 of 24 checks passed

bshaffer deleted the introduce-marketingplatformadmin branch

September 27, 2024 19:01

release-please bot mentioned this pull request

chore(main): release 0.261.0 #7698

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels