[buddypress-trac] [BuddyPress Trac] #7604: Merge WP-CLI commands into core
buddypress-trac
noreply at wordpress.org
Wed Nov 29 17:58:50 UTC 2017
#7604: Merge WP-CLI commands into core
--------------------------+-----------------------------
Reporter: boonebgorges | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: Core | Version:
Severity: normal | Resolution:
Keywords: |
--------------------------+-----------------------------
Comment (by boonebgorges):
Replying to [comment:5 DJPaul]:
> You think it's best to keep all these commands in a new folder, and not
put each WP-CLI command file into the existing component subfolders/naming
convention?
I don't know yet. I *don't* think it makes sense to think of CLI as
another "component" - components are already pretty ill-defined, and CLI
commands don't really even match that poor definition. So I think there
are two options:
1. introduce some sort of new organizational principle for stuff like this
- maybe something PSR-4 compliant, or whatever
2. put the individual CLI subcommand files in with their components
My only misgiving about 2 is that there are likely to be some parts of the
CLI suite that shared between components, or are not really linked to a
component. I suppose those could go in bp-core.
All told, I don't have a strong opinion. Whatever makes life easiest for
devs.
Worth noting that we'll probably want to make a similar decision when/if
we want to integrate some sort of API endpoints, so it may pay to think
ahead somewhat.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7604#comment:6>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list