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

docs: change properties with function value to methods #1715

Merged
merged 1 commit into from
Nov 18, 2021
Merged

Conversation

fhinkel
Copy link
Contributor

@fhinkel fhinkel commented Nov 18, 2021

Internally, see b/206943288

@fhinkel fhinkel requested review from a team as code owners November 18, 2021 02:52
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 18, 2021
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/nodejs-storage API. label Nov 18, 2021
@fhinkel fhinkel closed this Nov 18, 2021
@fhinkel fhinkel reopened this Nov 18, 2021
@fhinkel fhinkel closed this Nov 18, 2021
@fhinkel fhinkel changed the title test function vs method fix: change properties with function value to methods Nov 18, 2021
@fhinkel fhinkel reopened this Nov 18, 2021
@@ -629,7 +629,10 @@ class Bucket extends ServiceObject {
acl: Acl;
iam: Iam;

getFilesStream: (query?: GetFilesOptions) => Readable;
getFilesStream(query?: GetFilesOptions): Readable {
// placeholder body, overwritten in constructor
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems reasonble to me, but I would open a ticket in parser to fix this issue, and reference the ticket # here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You mean in api-documnter? Yeah, I can open an issue there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@shaffeeullah shaffeeullah left a comment

Choose a reason for hiding this comment

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

Does this PR have to be fix? Can it be docs?

@fhinkel
Copy link
Contributor Author

fhinkel commented Nov 18, 2021

Does this PR have to be fix? Can it be docs?

Sure!

@fhinkel fhinkel changed the title fix: change properties with function value to methods docs: change properties with function value to methods Nov 18, 2021
@shaffeeullah shaffeeullah self-requested a review November 18, 2021 17:49
@fhinkel fhinkel merged commit c365254 into main Nov 18, 2021
@fhinkel fhinkel deleted the fhinkel-methods branch November 18, 2021 18:01
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. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants