Skip to content

bump up package and axios version#48

Merged
QilongTang merged 3 commits intoDynamoDS:masterfrom
Enzo707:feature/update-package-version
Jan 10, 2024
Merged

bump up package and axios version#48
QilongTang merged 3 commits intoDynamoDS:masterfrom
Enzo707:feature/update-package-version

Conversation

@Enzo707
Copy link
Copy Markdown
Contributor

@Enzo707 Enzo707 commented Jan 9, 2024

Bumped up:

  • axios version to 1.6.5
  • package version to 0.0.27

Please review @QilongTang

@Enzo707 Enzo707 changed the title bump up package version to 0.0.27 and axios version to 1.6.5 bump up package and axios version Jan 9, 2024
@QilongTang
Copy link
Copy Markdown
Contributor

hi @Enzo707 What's up with so many changes in package-lock file? If you look at the other PR https://github.com/DynamoDS/NotificationCenter/pull/45/files there are much less changes there. If you have already tested the version bump, we could merge the other PR first then make this PR only about package version update

@Enzo707
Copy link
Copy Markdown
Contributor Author

Enzo707 commented Jan 10, 2024

hi @Enzo707 What's up with so many changes in package-lock file? If you look at the other PR https://github.com/DynamoDS/NotificationCenter/pull/45/files there are much less changes there. If you have already tested the version bump, we could merge the other PR first then make this PR only about package version update

@QilongTang My bad, I re-wrote the entire package-lock. I've fixed the PR with only the necessary changes.
Everything seems to work as expected with the new axios version

@QilongTang QilongTang merged commit 88e5327 into DynamoDS:master Jan 10, 2024
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.

2 participants