NotificationsYou must be signed in to change notification settings
Fork90
Star131

fix: remove use of old external mock#676

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

a-detiste wants to merge2 commits intogoogleapis:mainfroma-detiste:main

Closed

fix: remove use of old external mock#676

a-detiste wants to merge2 commits intogoogleapis:mainfroma-detiste:main

Conversation

Copy link

a-detiste commentedJul 6, 2024

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

Make sure to open an issue: there were various attempts before that got stuck by lack of CLA
Ensure the tests and linter pass
Code coverage does not decrease (if any source code was changed)
Appropriate docs were updated (if necessary)

a-detiste requested review froma team ascode owners

July 6, 2024 21:50

Copy link

conventional-commit-lint-gcfbot commentedJul 6, 2024•
edited
Loading

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or useautomerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/