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

refactor(commands): remove create commands #497

Merged
merged 1 commit into from
Feb 1, 2019
Merged

Conversation

eysi09
Copy link
Collaborator

@eysi09 eysi09 commented Jan 31, 2019

BREAKING CHANGE:

After this, the create project and create module commands will no
longer be available. We're removing them for now because currently
they're more confusing than they are useful. There's an open Github
issue (#312) for properly implementing the "create" functionality.

Closes #492.

@eysi09 eysi09 force-pushed the remove-create-command branch from 273c35c to 4b61498 Compare January 31, 2019 15:22
edvald
edvald previously approved these changes Jan 31, 2019
@eysi09 eysi09 force-pushed the remove-create-command branch from 4b61498 to 7ed46fe Compare February 1, 2019 09:08
@eysi09 eysi09 dismissed edvald’s stale review February 1, 2019 09:23

The docs still contain references to the create command.

BREAKING CHANGE:

After this, the `create project` and `create module` commands will no
longer be available. We're removing them for now because currently
they're more confusing than they are useful. There's an open Github
issue (#312) for properly implementing the "create" functionality.
@eysi09 eysi09 force-pushed the remove-create-command branch from 7ed46fe to 88d18d8 Compare February 1, 2019 09:35
@eysi09
Copy link
Collaborator Author

eysi09 commented Feb 1, 2019

@edvald I dismissed your review since I noticed that the docs mention the create commands. I've updated them accordingly.

@edvald edvald merged commit 386350d into master Feb 1, 2019
@edvald edvald deleted the remove-create-command branch February 1, 2019 13:31
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