Skip to content
This repository was archived by the owner on Sep 16, 2023. It is now read-only.

fix: Add shopt -s nullglob to dependencies script#611

Merged
dinagraves merged 1 commit into
masterfrom
autosynth-synthtool
Jul 2, 2021
Merged

fix: Add shopt -s nullglob to dependencies script#611
dinagraves merged 1 commit into
masterfrom
autosynth-synthtool

Conversation

@yoshi-automation

@yoshi-automation yoshi-automation commented Jun 20, 2021

Copy link
Copy Markdown
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/aef5e9f5-f459-4748-818b-f33209beded5/targets

  • To automatically regenerate this PR, check this box. (May take up to 24 hours.)

Source-Link: googleapis/synthtool@87254ac

@yoshi-automation
fix: Add shopt -s nullglob to dependencies script
7b050e2 
This prevents processing the unexpanded glob "**/.flattened-pom.xml" in repos which do not flatten any POMs.

Source-Author: dpcollins-google <[email protected]>
Source-Date: Sat Jun 19 18:35:58 2021 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 87254ac89a9559864c0a245d6b150406439ce3d8
Source-Link: googleapis/synthtool@87254ac
@yoshi-automation yoshi-automation requested review from a team June 20, 2021 08:37
@product-auto-label product-auto-label Bot added the api: containeranalysis Issues related to the googleapis/java-containeranalysis API. label Jun 20, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 20, 2021
@Neenu1995 Neenu1995 added the automerge Merge the pull request once unit tests and other checks pass. label Jun 22, 2021
@gcf-merge-on-green

gcf-merge-on-green Bot commented Jun 22, 2021

Copy link
Copy Markdown

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 22, 2021
@Neenu1995 Neenu1995 added the automerge Merge the pull request once unit tests and other checks pass. label Jun 23, 2021
@gcf-merge-on-green

gcf-merge-on-green Bot commented Jun 23, 2021

Copy link
Copy Markdown

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 23, 2021
@dinagraves dinagraves added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 2, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 2, 2021
@dinagraves dinagraves merged commit a256e57 into master Jul 2, 2021
@dinagraves dinagraves deleted the autosynth-synthtool branch July 2, 2021 18:51
@release-please release-please Bot mentioned this pull request Jan 4, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@Neenu1995 Neenu1995 Neenu1995 approved these changes

@dinagraves dinagraves dinagraves approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: containeranalysis Issues related to the googleapis/java-containeranalysis API. cla: yes This human has signed the Contributor License Agreement. context: full

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yoshi-automation @Neenu1995 @dinagraves @yoshi-kokoro