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

docs: fix cli param #459

Merged
merged 1 commit into from
Sep 22, 2023
Merged

docs: fix cli param #459

merged 1 commit into from
Sep 22, 2023

Conversation

ananni13
Copy link
Contributor

@ananni13 ananni13 commented Sep 22, 2023

--init is not a valid option for create-remix.

Launching the script as it is now, the CLI will pick up --init as the folder name to use to create the project, and won't ask for one.

If the intention was to automatically launch the remix.init script, we should use --init-script instead.

Test Plan

Checklist

- [ ] Tests updated

  • Docs updated

Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@kentcdodds kentcdodds merged commit 892b339 into epicweb-dev:main Sep 22, 2023
5 checks passed
@ananni13 ananni13 deleted the pr/docs-fix-cli-param branch September 22, 2023 14:32
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