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

Added doc for --copy-ignored flag #2151

Merged
merged 3 commits into from
Jan 10, 2020
Merged

Added doc for --copy-ignored flag #2151

merged 3 commits into from
Jan 10, 2020

Conversation

rajasekarm
Copy link
Member

@rajasekarm rajasekarm commented Jan 6, 2020

Documented --copy-ignored (babel/babel#10887) in options section as per @existentialism comment.

#2146 (comment)

@netlify
Copy link

netlify bot commented Jan 6, 2020

Deploy preview for babel ready!

Built with commit 09ea03a

https://deploy-preview-2151--babel.netlify.com

@netlify
Copy link

netlify bot commented Jan 6, 2020

Deploy preview for babel failed.

Built with commit 39e0236

https://app.netlify.com/sites/babel/deploys/5e15a093c7af0900089d4f2b

@JLHwung
Copy link
Contributor

JLHwung commented Jan 7, 2020

We do have a dedicated babel-cli docs, can you move it to docs/cli.md?

@rajasekarm
Copy link
Member Author

@JLHwung I moved here as per @existentialism comment. #2146 (comment)

@JLHwung
Copy link
Contributor

JLHwung commented Jan 8, 2020

Ah I see, the advanced section of cli.md refers to this page. I feel it’s more natural if we move cli options before option concepts, same as options falling into other categories.

@rajasekarm
Copy link
Member Author

@JLHwung Done

@nicolo-ribaudo nicolo-ribaudo merged commit b5981b7 into babel:master Jan 10, 2020
@rajasekarm rajasekarm deleted the copy-ignored-doc branch January 10, 2020 21:24
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

3 participants