Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New command to publish assets #1644

Closed
tobyzerner opened this issue Nov 14, 2018 · 2 comments · Fixed by #2731
Closed

New command to publish assets #1644

tobyzerner opened this issue Nov 14, 2018 · 2 comments · Fixed by #2731
Milestone

Comments

@tobyzerner
Copy link
Contributor

From b26eb8e:

Extract publishing assets from the migrate command into a new command. Possible names:

  • publish
  • vendor:publish (this is what Laravel uses right?)
  • asset(s):publish
  • ...

Also will need to consolidate the publishing logic with the installer while we're at it.

@tobyzerner tobyzerner added this to the 0.1.0-beta.9 milestone Nov 14, 2018
@luceos
Copy link
Member

luceos commented Nov 14, 2018

I am in favor of asset:publish.

  • Can we allow other extensions to hook into this command to be able to publish their extensions as well.
  • I've also felt that the way vendor:publish works with the --provider or --tag flag was pretty overcomplicated, although it suits the frameworks needs.
  • Maybe we can make this interactive once it grows too much by allowing the user to toggle what extension is allowed to publish?

@lubobill1990
Copy link
Contributor

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants