feat: bootstrap new command namespaces #632
Conversation
github-actions
bot
added
the
t-tooling
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request is neither linked to an issue or epic nor labeled as adhoc!
|
Bleh, labelling as adhoc because idk if I can link up 3 issues (and it solves only like 1% of each) |
| import { ApifyCommand } from '../../lib/apify_command.js'; | ||
|
|
||
| export class ActorIndexCommand extends ApifyCommand<typeof ActorIndexCommand> { | ||
| static override description = 'Commands are designed to be used with Actors.'; |
There was a problem hiding this comment.
Since this is an override, I guess it'll get glued to something else before printing to the console? On its own, the wording would be kinda weird.
There was a problem hiding this comment.
Yep, its printed when you run apify actors alone
There was a problem hiding this comment.
So what's the exact thing that gets printed in that case?
There was a problem hiding this comment.
When you have commands those get listed too
There was a problem hiding this comment.
Gotcha, thanks. Then I'd probably reword those descriptions along the lines of "Commands for working with Actors" or "for management of Actors"
There was a problem hiding this comment.
Thats more of a @netmilk task, I just did what we did for other namespaces. Agreed it could use better copies but I wouldn't wanna block feature implementation just bc of that
There was a problem hiding this comment.
I get it, but if we don't do it now, there will always be something more pressing than changing wording.
There was a problem hiding this comment.
Made #637 to track this
vladfrangu
changed the title
Prepares the following namespaces for commands: