Skip to content
This repository was archived by the owner on Apr 15, 2024. It is now read-only.

cortex skills generate command added#255

Merged
ljha-CS merged 11 commits intodevelopfrom
FAB-230
Dec 4, 2020
Merged

cortex skills generate command added#255
ljha-CS merged 11 commits intodevelopfrom
FAB-230

Conversation

@ljha-CS
Copy link
Copy Markdown
Collaborator

@ljha-CS ljha-CS commented Nov 12, 2020

Checklist:

Please check you fulfill ALL of the relevant checkboxes

  • Notified docs of any potential USER-facing changes
  • Added short description of the change - with relevant motivation and context.
  • Branch has the ticket number in its name (along with a ticket summary)
  • Commented the code, particularly in hard-to-understand areas
  • Added tests that prove my fix is effective or that my feature works
  • Ran npm test and it passes
  • Changes generate no new warnings

@jgielstra-cs
Copy link
Copy Markdown
Contributor

Do we need to keep generator cortex? Should we embed the template in cortex-cli to avoid this dep??

@ljha-CS
Copy link
Copy Markdown
Collaborator Author

ljha-CS commented Nov 18, 2020

Do we need to keep generator cortex? Should we embed the template in cortex-cli to avoid this dep??

Added templates in cli repo

@ljha-CS ljha-CS merged commit 657baff into develop Dec 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants