Skip to content

[GO] use latest available oauth2 version#8764

Merged
wing328 merged 2 commits intoOpenAPITools:masterfrom
acabarbaye:patch-1
Feb 24, 2021
Merged

[GO] use latest available oauth2 version#8764
wing328 merged 2 commits intoOpenAPITools:masterfrom
acabarbaye:patch-1

Conversation

@acabarbaye
Copy link
Copy Markdown
Contributor

@acabarbaye acabarbaye commented Feb 19, 2021

Temporary fix for #8763 till an official release of oauth2 package is actually made

Using the latest available version of this package at the time of writing https://pkg.go.dev/golang.org/x/oauth2?tab=versions

@wing328
Copy link
Copy Markdown
Member

wing328 commented Feb 23, 2021

Can you please update the samples so that the CI can verify the change?

  • Run the following to build the project and update samples:
    ./mvnw clean package 
    ./bin/generate-samples.sh
    ./bin/utils/export_docs_generators.sh
    
    Commit all changed files.
    This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
    These must match the expectations made by your contribution.
    You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/java*.
    For Windows users, please run the script in Git BASH.

@wing328
Copy link
Copy Markdown
Member

wing328 commented Feb 24, 2021

Pushed f27c819 to update the samples. Let's see how that goes.

@wing328
Copy link
Copy Markdown
Member

wing328 commented Feb 24, 2021

FYI. @antihax (2017/11) @grokify (2018/07) @kemokemo (2018/09) @jirikuncar (2021/01)

@wing328 wing328 merged commit a31dba5 into OpenAPITools:master Feb 24, 2021
@acabarbaye acabarbaye deleted the patch-1 branch March 5, 2021 12:16
jirikuncar added a commit to DataDog/datadog-api-client-go that referenced this pull request Mar 11, 2021
@wing328
Copy link
Copy Markdown
Member

wing328 commented Mar 22, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants