FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: update docs/dyn by yoshi-automation · Pull Request #1137 · googleapis/google-api-python-client · GitHub

chore: update docs/dyn - #1137

Closed
yoshi-automation wants to merge 3 commits into
masterfrom
autosynth-self
Closed

chore: update docs/dyn#1137
yoshi-automation wants to merge 3 commits into
masterfrom
autosynth-self

Conversation

Copy link
Copy Markdown
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/f632d5d1-d20c-4ae6-90f5-18e53f0f588a/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/discovery-artifact-manager@0bed8bd
Source-Link: googleapis/synthtool@c2de321

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.
Source-Author: Daniel Azuma <dazuma@google.com>
Source-Date: Tue Nov 24 09:22:42 2020 -0800
Source-Repo: googleapis/synthtool
Source-Sha: c2de32114ec484aa708d32012d1fa8d75232daf5
Source-Link: googleapis/synthtool@c2de321
Source-Author: Google APIs <googleapis-publisher@google.com>
Source-Date: Wed Nov 25 07:52:04 2020 -0800
Source-Repo: googleapis/discovery-artifact-manager
Source-Sha: 0bed8bdae25d545e796cfcdd7a9bfffff11e69da
Source-Link: googleapis/discovery-artifact-manager@0bed8bd
yoshi-automation requested a review from a team December 15, 2020 13:19
google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 15, 2020
busunkim96 changed the title [CHANGE ME] Re-generated to pick up changes from self. chore: update docs/dyn Dec 15, 2020
<p class="firstline">Gets an approval request. Returns NOT_FOUND if the request does not exist.</p>
<p class="toc_element">
<code><a href="#list">list(parent, filter=None, pageToken=None, pageSize=None, x__xgafv=None)</a></code></p>
<code><a href="#list">list(filter=None, pageSize=None, pageToken=None, parent, x__xgafv=None)</a></code></p>

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@parthea It looks like the code might need to be adjusted to keep track of positional v. named args.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Good catch! I'll take a look.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Will be fixed in #1141

busunkim96 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Dec 15, 2020
parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Dec 16, 2020
parthea closed this Dec 16, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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. context: partial

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL