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

fix(owl-bot): ignore non-cloud php repos #5286

Merged
merged 1 commit into from
Feb 7, 2024
Merged

fix(owl-bot): ignore non-cloud php repos #5286

merged 1 commit into from
Feb 7, 2024

Conversation

chingor13
Copy link
Contributor

@chingor13 chingor13 commented Feb 7, 2024

owl-bot is opening some PRs against sharded, non-php repos. We should not be opening copy-code PRs against these repos.

example: googleapis/php-shopping-merchant-reports#5

These repos googleapis/php-<something>, are non-cloud and split of the primary PHP monorepo googleapis/google-cloud-php.

@chingor13 chingor13 requested a review from a team as a code owner February 7, 2024 00:39
@chingor13 chingor13 requested a review from sofisl February 7, 2024 00:42
@chingor13
Copy link
Contributor Author

Note that this won't close existing open PRs, but should prevent new PRs from being opened (after the scan GitHub configs cron task runs)

Copy link
Contributor

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

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

LGTM, I have verified all existing googleapis/php-* repos are subrepos and should be ignored. The only non-monorepo owlbot repo is googleapis/common-protos-php (which at some point we should move under googleapis/google-cloud-php).

@chingor13 chingor13 enabled auto-merge (squash) February 7, 2024 00:55
@@ -32,6 +32,6 @@ export function shouldIgnoreRepo(ownerSlashRepo: string): boolean {
// subdirectories of google-cloud-php and never accept pull requests.
const regexp = process.env.IGNORE_REPO_REGEXP
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like these should always be ignored, not only if they're not explicitly specified.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think is an override for tests/testing environments and we use the default in production

@sofisl sofisl self-requested a review February 7, 2024 20:01
@chingor13 chingor13 merged commit 1700f8e into main Feb 7, 2024
22 checks passed
@chingor13 chingor13 deleted the ignore-php branch February 7, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants