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 backoff/v3 to backoff/v4 #26868

Merged
merged 2 commits into from
May 29, 2024
Merged

Bump backoff/v3 to backoff/v4 #26868

merged 2 commits into from
May 29, 2024

Conversation

crazybolillo
Copy link
Contributor

This makes it so projects that are already using backoff don't need to pull up an old version of it just because of vault/api. This major version change is compatible with existing usage in code.

Refs #24707.

Copy link

hashicorp-cla-app bot commented May 7, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@heatherezell heatherezell added the dependencies Pull requests that update a dependency file label May 7, 2024
@VioletHynes
Copy link
Contributor

Hey @crazybolillo ! I'm interested in getting this merged. I just tested this a little locally and it doesn't look like it breaks anything.

Before we can do that, it looks like there's some merge conflicts. Would it be possible for you to resolve them? Once those get resolved, I'd be happy to approve and merge this in.

This makes it so projects that are already using backoff don't need to
pull up an old version of it just because of vault/api. This major
version change is compatible with existing usage in code.

Refs hashicorp#24707.
@crazybolillo
Copy link
Contributor Author

Thanks, I have rebased the MR and fixed the conflicts.

Copy link
Contributor

@VioletHynes VioletHynes left a comment

Choose a reason for hiding this comment

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

This looks good to me! I'll wait for CI to run and for everything to be green, and I'll try to get it merged in after.

@VioletHynes VioletHynes merged commit fab2899 into hashicorp:main May 29, 2024
65 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants