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

feat(auth): samples for auth #2759

Closed
wants to merge 8 commits into from
Closed

feat(auth): samples for auth #2759

wants to merge 8 commits into from

Conversation

FrodoTheTrue
Copy link
Contributor

No description provided.

@FrodoTheTrue FrodoTheTrue requested a review from a team as a code owner September 16, 2022 10:25
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Sep 16, 2022
@snippet-bot
Copy link

snippet-bot bot commented Sep 16, 2022

Here is the summary of changes.

You are about to add 6 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@kweinmeister kweinmeister requested a review from a team as a code owner October 11, 2022 22:00
@grayside
Copy link
Collaborator

grayside commented Nov 8, 2022

Is this PR migrating samples from https://github.com/googleapis/google-auth-library-nodejs/tree/main/samples? If so could we follow the revised process to keep the Git commit history?

@rsamborski
Copy link
Member

@Shnatsel Can you PTAL and get this merged?
@Sita04 can provide more context

Copy link
Contributor

@ace-n ace-n left a comment

Choose a reason for hiding this comment

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

Highlighting @grayside's comment:

Is this PR migrating samples from https://github.com/googleapis/google-auth-library-nodejs/tree/main/samples? If so could we follow the revised process to keep the Git commit history?

@FrodoTheTrue FYI: it's probably easiest to close this PR and create a new one.

(Feel free to DM me internally or post in the (internal) migration chat channel if any of this is unclear! 😃)

@kweinmeister kweinmeister requested a review from a team as a code owner December 19, 2022 18:36
@Shnatsel
Copy link
Contributor

The samples have instead been added to the client library repository. Since the client library is not automatically generated, there are currently no plans to move the samples to nodejs-docs-samples. Closing.

@Shnatsel Shnatsel closed this Dec 23, 2022
@pattishin pattishin deleted the add-auth-samples branch June 30, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants