-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: support spaced commands #166
Conversation
test/hooks/not-found.test.ts
Outdated
@@ -17,6 +17,17 @@ describe('command_not_found', () => { | |||
expect(ctx.stdout).to.be.contain('commands\nhelp') | |||
}) | |||
|
|||
test | |||
.only() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like a leftover .only
... not super familiar with OCLIF but lgtm
404d4b1
to
6437d30
Compare
}, | ||
"engines": { | ||
"node": ">=8.0.0" | ||
"node": ">=14.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm starting to second guess this. Since node 12 is still a LTS until mid-2022, I wonder if it is a good move to require 14. I guess another way to ask this, is it ok the unified CLI requires node 14? Maybe that is... we can also see what kind of reaction we get when we do the announcement post.
.circleci/config.yml
Outdated
- release: | ||
context: org-global | ||
filters: | ||
branches: {only: master} | ||
branches: {only: main} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing we aren't merging any of these PRs until we have an @oclif/core@v1
?
b90032c
to
aea5dd7
Compare
* feat: support spaced commands * fix: do not suggest hidden commands * chore: prepare for v2 * chore: update core * chore: update circle config * chore: remove github actions * chore: regenerate lock file
Adds support for spaced commands
Depends on oclif/core#143
Fixes #28