Skip to content

Conversation

@thiyaguk09
Copy link
Contributor

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 as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes

thiyaguk09 and others added 22 commits September 6, 2024 15:26
…elds'-to-getFiles

feat(storage):  add support for 'fields' query parameter to getFiles
…elds'-to-getFiles

build: fix path-to-regexp to older version due to node 14 requirement
…elds'-to-getFiles

Remove unnecessary sample code
Node Transfer Manager: add support for 'skipIfExists' option for
downloadMany
…dMany-in-transfer-manager

feat(storage): add support for 'skipIfExists' option for downloadMany
…okoro

feat: adds integration tests for Universe Domain configuration with
kokoro changes
…-tests

feat: adds integration tests for Universe Domain configuration
@thiyaguk09 thiyaguk09 requested review from a team as code owners September 30, 2024 06:51
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Sep 30, 2024
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • .kokoro/continuous/node14/system-test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/presubmit/node14/system-test.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/release/publish.cfg - .kokoro files are templated and should be updated in synthtool
  • .kokoro/system-test.sh - .kokoro files are templated and should be updated in synthtool

@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/nodejs-storage API. label Sep 30, 2024
@thiyaguk09 thiyaguk09 added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 30, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 30, 2024
@ddelgrosso1
Copy link
Contributor

Code looks good, think the after hook needs some adjustment as it is failing.

@thiyaguk09
Copy link
Contributor Author

Code looks good, think the after hook needs some adjustment as it is failing.

The after hook has been adjusted. Please review the changes.

@ddelgrosso1 ddelgrosso1 added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 3, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 3, 2024
@thiyaguk09 thiyaguk09 requested a review from a team as a code owner October 3, 2024 13:38
@thiyaguk09 thiyaguk09 requested a review from ddelgrosso1 October 3, 2024 14:02
@ddelgrosso1 ddelgrosso1 added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 3, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 3, 2024
Copy link
Contributor

@ddelgrosso1 ddelgrosso1 left a comment

Choose a reason for hiding this comment

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

What happened to the kokoro changes? I saw them at one point.

@thiyaguk09
Copy link
Contributor Author

What happened to the kokoro changes? I saw them at one point.

gcf-owl-bot removed those changes 31ca744.

@thiyaguk09 thiyaguk09 requested a review from ddelgrosso1 October 4, 2024 13:38
@ddelgrosso1 ddelgrosso1 added the owlbot:ignore instruct owl-bot to ignore a PR label Oct 4, 2024
@ddelgrosso1 ddelgrosso1 added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 7, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 7, 2024
@ddelgrosso1 ddelgrosso1 merged commit 53db6ba into googleapis:main Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/nodejs-storage API. owlbot:ignore instruct owl-bot to ignore a PR size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants