Skip to content

fix(internal/gapicgen): filter out internal directory changes#4085

Merged
codyoss merged 3 commits intogoogleapis:masterfrom
codyoss:filter-internal
May 12, 2021
Merged

fix(internal/gapicgen): filter out internal directory changes#4085
codyoss merged 3 commits intogoogleapis:masterfrom
codyoss:filter-internal

Conversation

@codyoss
Copy link
Copy Markdown
Member

@codyoss codyoss commented May 11, 2021

  • Filter out internal dir changes since they do not need a module
    update.
  • Remove unneeded param from method
  • Log extra details when a excuting a command yields an error

- Filter out internal dir changes since they do not need a module
  update.
- Remove unneeded param from method
- Log extra details when a excuting a command yields an error
@codyoss codyoss requested a review from a team May 11, 2021 17:02
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label May 11, 2021
@codyoss codyoss requested a review from tbpg May 11, 2021 17:02
@codyoss
Copy link
Copy Markdown
Member Author

codyoss commented May 12, 2021

@noahdietz Can I get you to take a peek at this PR. It fixes a bug that is currently blocking regen.

@codyoss codyoss merged commit 01473f6 into googleapis:master May 12, 2021
@codyoss codyoss deleted the filter-internal branch May 12, 2021 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants