Skip to content

fix: add serve to internalCommand#3451

Merged
lex111 merged 1 commit into
facebook:masterfrom
longlho:patch-1
Sep 23, 2020
Merged

fix: add serve to internalCommand#3451
lex111 merged 1 commit into
facebook:masterfrom
longlho:patch-1

Conversation

@longlho

@longlho longlho commented Sep 15, 2020

Copy link
Copy Markdown
Contributor

Motivation

Right now docusaurus serve my-dir doesn't work because it's not registered as an internal command, thus always forced to invoked w/ path.resolve('.'). This fixes that

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Manually verified in formatjs

Related PRs

Right now `docusaurus serve my-dir` doesn't work because it's not registered as an internal command, thus always forced to invoked w/ `path.resolve('.')`. This fixes that
@longlho longlho requested a review from yangshun as a code owner September 15, 2020 02:43
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Sep 15, 2020
@docusaurus-bot

Copy link
Copy Markdown
Contributor

Deploy preview for docusaurus-2 ready!

Built without sensitive environment variables with commit 8b74dac

https://deploy-preview-3451--docusaurus-2.netlify.app

@longlho

longlho commented Sep 21, 2020

Copy link
Copy Markdown
Contributor Author

ping @yangshun

@slorber

slorber commented Sep 23, 2020 via email

Copy link
Copy Markdown
Collaborator

@lex111 lex111 merged commit 407581d into facebook:master Sep 23, 2020
@lex111

lex111 commented Sep 23, 2020

Copy link
Copy Markdown
Contributor

I think it's ready for the merge now.
@longlho thank you!

@lex111 lex111 added this to the v2.0.0-alpha.65 milestone Sep 26, 2020
slorber pushed a commit that referenced this pull request Sep 29, 2020
Right now `docusaurus serve my-dir` doesn't work because it's not registered as an internal command, thus always forced to invoked w/ `path.resolve('.')`. This fixes that
mrizwanashiq pushed a commit to mrizwanashiq/docusaurus that referenced this pull request Jun 25, 2026
Right now `docusaurus serve my-dir` doesn't work because it's not registered as an internal command, thus always forced to invoked w/ `path.resolve('.')`. This fixes that
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants