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 dependencies #60

Merged
merged 1 commit into from
May 5, 2023
Merged

Conversation

courcelan
Copy link
Contributor

upgrade axios to 1.3.6, webpack to 5.81.0

@courcelan courcelan added the dependencies Pull requests that update a dependency file label Apr 27, 2023
@ViolanteCodes
Copy link
Contributor

@courcelan should we do a version bump and changelog on this?

@courcelan
Copy link
Contributor Author

courcelan commented May 1, 2023 via email

@ViolanteCodes ViolanteCodes force-pushed the chore/upgrade-dependencies-04_29_2023 branch 2 times, most recently from a8e6d03 to 4e6718e Compare May 1, 2023 16:49
@ViolanteCodes
Copy link
Contributor

@courcelan i had to rebase due to conflicts with the previous merged PR - I think this okay (installs good, tests are passing), but can you take another look just to make sure?

@ViolanteCodes
Copy link
Contributor

@courcelan
Copy link
Contributor Author

Hi @ViolanteCodes, just re-ran yarn to get the right lockfile into place, but all looks good from here. Thanks

@ViolanteCodes ViolanteCodes force-pushed the chore/upgrade-dependencies-04_29_2023 branch from 2b4251b to fb975a2 Compare May 5, 2023 13:16
Copy link
Contributor

@ViolanteCodes ViolanteCodes left a comment

Choose a reason for hiding this comment

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

LGTM

@ViolanteCodes ViolanteCodes merged commit ecaf5a7 into master May 5, 2023
@ViolanteCodes ViolanteCodes deleted the chore/upgrade-dependencies-04_29_2023 branch May 5, 2023 13:17
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.

2 participants