Page MenuHomePhabricator

Added project option and default to todo workflow
Needs RevisionPublic

Authored by svenax on Oct 30 2013, 2:10 PM.
Tags
None
Referenced Files
F14007457: D7446.diff
Tue, Oct 29, 6:11 AM
F14001235: D7446.diff
Fri, Oct 25, 5:28 AM
F13995109: D7446.id.diff
Wed, Oct 23, 10:42 AM
F13986866: D7446.diff
Mon, Oct 21, 6:01 AM
F13959290: D7446.diff
Mon, Oct 14, 6:44 PM
Unknown Object (File)
Jul 25 2024, 4:06 AM
Unknown Object (File)
Jul 7 2024, 12:13 AM
Unknown Object (File)
Jul 6 2024, 11:45 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary
  • Option to add a todo task to one or more projects.
  • Read default project from todo.default_project.
Test Plan
  • arc todo Test test --proj someproject
  • arc todo Test test with todo.default_project set

Diff Detail

Branch
todo-project-option
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

  • I think you should be able to specify more than one default project -- a list rather than a single item.
  • Although not all of it is consistent, we're moving toward hyphens over underscores in config. So a better name would be arc.todo.default-projects.
  • Config entries should be listed in ArcanistSettings so that the "list config" commands know about them and they can be typechecked.
  • Making this context sensitive seems like it might be confusing? It's a little weird to me that you pick up projects based on which directory you run arc todo in. It's also not obvious to me what the interaction between -p and defaults should be. This implements "replace defaults with flags", but maybe "add flags to defaults" is more intuitive? Generally, I'm not sure about the value of the default setting.
  • For consistency, don't abbreviate the full-length flag -- --project, not --proj. The short flag is better if you don't want to type --project anyway.
  • For consistency, shorten to -p instead of -P?
  • Projects will have proper short names soon (see T4021) so users could do arc todo -p qa instead of arc todo -p "Quality Assurance". Maybe this feature should wait for that?

Let me know how much of that makes sense.

src/workflow/ArcanistTodoWorkflow.php
95–99

This will tremendously overfetch project data. It would be better if we could query for just the projects the user has specified.