Skip to content
This repository was archived by the owner on Mar 3, 2026. It is now read-only.

feat(storage): add support for 'fields' query parameter to getFiles#2521

Merged
ddelgrosso1 merged 13 commits into
googleapis:mainfrom
thiyaguk09:main
Sep 17, 2024
Merged

feat(storage): add support for 'fields' query parameter to getFiles#2521
ddelgrosso1 merged 13 commits into
googleapis:mainfrom
thiyaguk09:main

Conversation

@thiyaguk09

Copy link
Copy Markdown
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 #2138

@thiyaguk09 thiyaguk09 requested review from a team September 17, 2024 08:11
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: storage Issues related to the googleapis/nodejs-storage API. labels Sep 17, 2024
Comment thread system-test/common.ts

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We adjusted the test placement to address the previous failures.

Comment thread package.json
"proxyquire": "^2.1.3",
"sinon": "^18.0.0",
"nise": "6.0.0",
"path-to-regexp": "6.2.2",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's just add a tacking issue to remove this once we upgrade to Node 18 minimum so we don't forget.

@ddelgrosso1 ddelgrosso1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ddelgrosso1 ddelgrosso1 added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2024
@ddelgrosso1 ddelgrosso1 merged commit f78fe92 into googleapis:main Sep 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/nodejs-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: add support for 'fields' query parameter to getFiles

2 participants