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

fix(deps): upgrade moment to 2.29.4 #545

Closed
wants to merge 1 commit into from

Conversation

roggervalf
Copy link
Collaborator

@roggervalf roggervalf commented Aug 10, 2022

Changes Made

Upgrade moment to version 2.29.4 ref #540, this one also changes the package in package.json file

Potential Risks

None

Test Plan

As it is a minor change version it should work as expected

Checklist

  • I've increased test coverage
  • Since this is a public repository, I've checked I'm not publishing private data in the code, commit comments, or this PR.

@roggervalf
Copy link
Collaborator Author

please @bradvogel when you get some chance

@bradvogel
Copy link
Contributor

Unfortunately we're not able to review farther fixes in this repo. I filed #546.

@roggervalf do you have an interest in being a maintainer?

@roggervalf
Copy link
Collaborator Author

hey @bradvogel, sure I can

@bradvogel
Copy link
Contributor

@roggervalf I granted you 'Maintain' access, so you should be able to review PRs from now on. I'll approve this one.

@bradvogel bradvogel self-requested a review August 11, 2022 18:05
@bradvogel
Copy link
Contributor

@roggervalf Actually, could we merge #540 instead? We don't need the update to package.json, as it uses the ^ syntax to specify the "latest minor version"

@roggervalf
Copy link
Collaborator Author

sure, it should be fine let me close this one

@roggervalf roggervalf closed this Aug 11, 2022
@roggervalf roggervalf deleted the fix-vulnerabilities branch August 11, 2022 20:08
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