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

Upgrade axios to 0.21.4 #107

Closed
wants to merge 1 commit into from
Closed

Upgrade axios to 0.21.4 #107

wants to merge 1 commit into from

Conversation

macsj200
Copy link

@macsj200 macsj200 commented Sep 9, 2021

There is a dependabot PR in forum-service to bump axios to 0.21.2. This PR is failing unit tests for some reason. I am hoping that upgrading past 0.21.2 to the latest 0.21.4 resolves these issues. Unfortunately the type signatures for the latest version of axios and the version of axios included in @lifeomic/alpha don't match.

This PR bumps axios to the latest version.

See:
https://github.com/lifeomic/forum-service/pull/171
https://github.com/lifeomic/life-extend/pull/1690
axios/axios#4036

There is a dependabot PR in forum-service to bump axios to `0.21.2`. This PR is failing unit tests for some reason. I am hoping that upgrading past `0.21.2` to the latest `0.21.4` resolves these issues. Unfortunately the type signatures for the latest version of axios and the version of axios included in `@lifeomic/alpha` don't match.

This PR bumps axios to the latest version.

See:
https://github.com/lifeomic/forum-service/pull/171
https://github.com/lifeomic/life-extend/pull/1690
axios/axios#4036
@macsj200 macsj200 closed this Sep 9, 2021
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.

None yet

1 participant