Skip to content
This repository was archived by the owner on Mar 27, 2024. It is now read-only.

v2 CLI (Alpha)#210

Closed
paganotoni wants to merge 92 commits into
mainfrom
v2
Closed

v2 CLI (Alpha)#210
paganotoni wants to merge 92 commits into
mainfrom
v2

Conversation

@paganotoni

@paganotoni paganotoni commented Aug 31, 2022

Copy link
Copy Markdown
Member

This PR changes the approach we've been taking on the CLI aiming to provide the ability to plugin custom commands to the Buffalo CLI, at a higher level this would compact the stack that will be supported by the Buffalo team and allow Buffalo developers to write plugins that solve their workflow specific needs.

In terms of the ability to override the CLI there are 2 main needs:

  • Allowing to add plugins on a Buffalo codebase
  • Allowing the use of custom plugins to generate Buffalo applications

These two capacities are vital to improving the out-of-the-box experience for Buffalo by allowing developers to provide support for their own tools and let the Buffalo maintainers focus on a better experience for the frontend stack.

Also, Some of the resulting effects of this new shape of the Buffalo CLI are:

Reducing third-party dependencies (using more of the standard lib)

  • Cleaning up old issues
  • Improving the maintainability of the Buffalo CLI.

In order to achieve those 2 overall goals we should complete the following steps:

  • replace help instructions
  • convert base commands to plugins
  • rewrite pop plugin
  • add plugins command (to list plugins)
  • implement plugin override API (remove, add, clear)
  • implement custom CLI main feature (in cmd/buffalo)
  • implement global per-user default plugins (in $HOME/.buffalo/cmd).

@paganotoni paganotoni requested a review from a team as a code owner August 31, 2022 14:51
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment. Otherwise, this will be closed in 7 days.

@github-actions github-actions Bot added the stale label Oct 21, 2022
@sio4 sio4 removed the stale label Oct 21, 2022
@paganotoni

Copy link
Copy Markdown
Member Author

After some time thinking about this, some conversations with @sio4 (thanks!) and a lot of effort I've decided to close this PR as the amount of effort to get this done may be something we better spend on improving the DX of the framework through other optimizations and not redoing the CLI.

@paganotoni paganotoni closed this Nov 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants