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

Bump yup-oauth2 to v10 #510

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davidkuridza
Copy link

Version 10 of yup-oauth2 was released recently, this PR bumps the version here as well.

@davidkuridza
Copy link
Author

Not sure what exactly is wrong, will try to look into it a bit later. Please don't hesitate to chip in :)

@Byron
Copy link
Owner

Byron commented Jun 12, 2024

Thanks for this!

You can take a look at the sibling PR for a fix.

@Byron Byron linked an issue Jun 12, 2024 that may be closed by this pull request
@davidkuridza
Copy link
Author

Thanks for this!

You can take a look at the sibling PR for a fix.

I rebased against the latest mainline and looked at #509, but can't figure it out. What am I missing here?

@Byron
Copy link
Owner

Byron commented Jun 15, 2024

Apologies, I could have a been a bit less cryptic. If you search for "yup-oauth" in etc/ there should be one or two yaml files that also need to be updated. With that changed, CI should be green.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to Hyper 1.0
2 participants