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 axios to stable release version #1990

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ttytm
Copy link

@ttytm ttytm commented Aug 19, 2024

The PR bumps axios to it's stable version. The major v1 was released in October 2022 and is where development and fixes are continued.

This resolves that it is currently not included in regular package updates but being stuck at the old development version due to the major version difference.

@CLAassistant
Copy link

CLAassistant commented Aug 19, 2024

CLA assistant check
All committers have signed the CLA.

@ttytm
Copy link
Author

ttytm commented Aug 23, 2024

@turbo124/@beganovich I'm seeing the other PRs mention you for a review.

A review for this one here likely comes with just little effort.

If it's better to rebase the request on the dev branch, please let me know, I wasn't sure about destination you prefer.

@beganovich
Copy link
Member

Hey there, thanks for your contribution. Can you please change the target to develop?

Thanks.

@ttytm ttytm changed the base branch from main to develop August 23, 2024 12:01
@ttytm ttytm marked this pull request as draft August 23, 2024 12:04
@ttytm

This comment was marked as outdated.

@ttytm ttytm marked this pull request as ready for review August 23, 2024 12:11
@beganovich
Copy link
Member

Before merging can you please verify if we cover all these cases? https://github.com/bmuenzenmeyer/axios-1.0.0-migration-guide

@ttytm
Copy link
Author

ttytm commented Aug 27, 2024

@beganovich thanks for taking care.

After checking again with linked migration guide, I don't see any caveats. So it should be good 👍

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.

3 participants