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 from 0.27.2 to 1.1.3 in /dashboard #5475

Merged
merged 4 commits into from Oct 24, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 18, 2022

Bumps axios from 0.27.2 to 1.1.3.

Release notes

Sourced from axios's releases.

v1.1.3

Added

Added custom params serializer support #5113

Fixed

Fixed top-level export to keep them in-line with static properties #5109 Stopped including null values to query string. #5108 Restored proxy config backwards compatibility with 0.x #5097 Added back AxiosHeaders in AxiosHeaderValue #5103 Pin CDN install instructions to a specific version #5060 Handling of array values fixed for AxiosHeaders #5085

Chores

docs: match badge style, add link to them #5046 chore: fixing comments typo #5054 chore: update issue template #5061 chore: added progress capturing section to the docs; #5084

Contributors to this release

v1.1.2

Fixed

  • Fixed broken exports for UMD builds.

Contributors to this release

v1.1.1

Fixed

  • Fixed broken exports for common js. This fix breaks a prior fix, I will fix both issues ASAP but the commonJS use is more impactful.

Contributors to this release

... (truncated)

Changelog

Sourced from axios's changelog.

[1.1.3] - 2022-10-15

Added

Added custom params serializer support #5113

Fixed

Fixed top-level export to keep them in-line with static properties #5109 Stopped including null values to query string. #5108 Restored proxy config backwards compatibility with 0.x #5097 Added back AxiosHeaders in AxiosHeaderValue #5103 Pin CDN install instructions to a specific version #5060 Handling of array values fixed for AxiosHeaders #5085

Chores

docs: match badge style, add link to them #5046 chore: fixing comments typo #5054 chore: update issue template #5061 chore: added progress capturing section to the docs; #5084

Contributors to this release

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Oct 18, 2022
@netlify
Copy link

netlify bot commented Oct 18, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit 115521a
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/634f36b68aaee9000996696b

antgamdia
antgamdia previously approved these changes Oct 18, 2022
Bumps [axios](https://github.com/axios/axios) from 0.27.2 to 1.1.3.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md)
- [Commits](axios/axios@v0.27.2...v1.1.3)

---
updated-dependencies:
- dependency-name: axios
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/dashboard/axios-1.1.3 branch from 6796f40 to 5fe9291 Compare October 18, 2022 10:53
@antgamdia
Copy link
Contributor

There are some incompatible changes, but, the main issue is that we are using "moxios" for mocking axios request and this project seems to be inactive (see axios/moxios#98). Maybe we will have to migrate to https://www.npmjs.com/package/axios-mock-adapter

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>

Conflicts:
	dashboard/package.json
@antgamdia antgamdia self-requested a review October 18, 2022 23:28
@antgamdia antgamdia dismissed their stale review October 18, 2022 23:29

dep update with manual intervention

@antgamdia antgamdia requested review from castelblanque and removed request for antgamdia October 18, 2022 23:29
@castelblanque
Copy link
Collaborator

castelblanque commented Oct 24, 2022

Last PR on Moxios dates back to March, 2021. I agree on migrating to another tool like https://www.npmjs.com/package/axios-mock-adapter or similar.

@antgamdia
Copy link
Contributor

Last PR on Moxios dates back to March, 2021. I agree on migrating to another tool like https://www.npmjs.com/package/axios-mock-adapter or similar.

Exactly, that's what I've done in this PR indeed :P and why I asked for a review

Copy link
Collaborator

@castelblanque castelblanque left a comment

Choose a reason for hiding this comment

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

Ok...sorry seeing the Bump axios from... title on the PR was confusing.
LGTM.

@antgamdia antgamdia merged commit 4ad748a into main Oct 24, 2022
@antgamdia antgamdia deleted the dependabot/npm_and_yarn/dashboard/axios-1.1.3 branch October 24, 2022 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-not-required dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants