-
Notifications
You must be signed in to change notification settings - Fork 437
Update fill command to have sane default execution #1610
Copy link
Copy link
Open
Labels
A-test-fillArea: execution_testing.cli.pytest_commands.plugins.fillerArea: execution_testing.cli.pytest_commands.plugins.fillerC-featCategory: an improvement or new featureCategory: an improvement or new featureE-easyExperience: easy, good for newcomersExperience: easy, good for newcomersP-lowS-needs-discussionStatus: needs discussionStatus: needs discussionstaleThe Issue/PR has not had any activity for 60 days. PRs will be automatically closed.The Issue/PR has not had any activity for 60 days. PRs will be automatically closed.
Milestone
Description
What was wrong?
When fill is called without any arguments it fails because there is no target directory or sane flags. This should not be the case.
How can it be fixed?
At the very least, calling fill without any arguments should print a usage message and exit. In the best case (IMO) it should run some default configuration that strikes a balance between commonality of use and execution time.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-test-fillArea: execution_testing.cli.pytest_commands.plugins.fillerArea: execution_testing.cli.pytest_commands.plugins.fillerC-featCategory: an improvement or new featureCategory: an improvement or new featureE-easyExperience: easy, good for newcomersExperience: easy, good for newcomersP-lowS-needs-discussionStatus: needs discussionStatus: needs discussionstaleThe Issue/PR has not had any activity for 60 days. PRs will be automatically closed.The Issue/PR has not had any activity for 60 days. PRs will be automatically closed.