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

Update dependencies #816

Merged
merged 1 commit into from Oct 5, 2021
Merged

Update dependencies #816

merged 1 commit into from Oct 5, 2021

Conversation

XhmikosR
Copy link
Contributor

@XhmikosR XhmikosR commented Oct 4, 2021

Nothing too fancy, but since strip-ansi has a security issue, we better force the update.

I'm going to keep this as draft, but ping me to update it before merging or a new release.

@XhmikosR
Copy link
Contributor Author

XhmikosR commented Oct 4, 2021

@xzyfer did you see my comments is #815? If you want to add a lock file I could add it here and adapt #813 too.

@xzyfer
Copy link
Collaborator

xzyfer commented Oct 4, 2021 via email

@XhmikosR
Copy link
Contributor Author

XhmikosR commented Oct 4, 2021

Done. After merging this, I'll update the other PRs to make use of npm ci and cache.

@XhmikosR XhmikosR marked this pull request as ready for review October 4, 2021 13:11
@xzyfer
Copy link
Collaborator

xzyfer commented Oct 5, 2021

Thanks

@xzyfer xzyfer merged commit e09ca1c into dlmanning:master Oct 5, 2021
@XhmikosR XhmikosR deleted the deps branch October 6, 2021 05:38
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

2 participants