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 Acorn and Minimist #79

Merged
merged 1 commit into from
Apr 2, 2020
Merged

Bump Acorn and Minimist #79

merged 1 commit into from
Apr 2, 2020

Conversation

meganyin13
Copy link
Member

Fix the security vulnerability.
My changes in #78 were merged in about 2 hours ago but the changes still haven't shown up on the main site. I'm not sure if this is the issue.

@meganyin13 meganyin13 requested a review from SamChou19815 April 2, 2020 01:31
@meganyin13 meganyin13 merged commit fed7120 into master Apr 2, 2020
@meganyin13 meganyin13 deleted the security-vulnerability branch April 2, 2020 01:36
@SamChou19815
Copy link
Contributor

@meganyin13 deploy command seems to be broken in the new docusaurus version.
See facebook/docusaurus#2496

@meganyin13
Copy link
Member Author

Should I wait for that PR to get merged in?

@SamChou19815
Copy link
Contributor

@meganyin13 you should probably downgrade docusaurus to alpha-48 for now :(

@yangshun
Copy link

yangshun commented Apr 2, 2020

Sorry about that. We'll publish hotfix today

SamChou19815 added a commit that referenced this pull request Apr 2, 2020
alpha.50 should have fixed the problem mentioned in #79.
@SamChou19815 SamChou19815 mentioned this pull request Apr 2, 2020
SamChou19815 added a commit that referenced this pull request Apr 2, 2020
alpha.50 should have fixed the problem mentioned in #79.
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.

3 participants