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

[3.0] Rename asset command to publish #694

Closed
wants to merge 1 commit into from
Closed

[3.0] Rename asset command to publish #694

wants to merge 1 commit into from

Conversation

dwightwatson
Copy link
Contributor

@dwightwatson dwightwatson commented Oct 24, 2019

I noticed that Horizon is the odd one out when it comes to the naming the command to re-publish assets.

roomies git:(master) ✗ php artisan | grep publish
  horizon:assets       Re-publish the Horizon assets
  nova:publish         Publish all of the Nova resources
  telescope:publish    Publish all of the Telescope resources
  vendor:publish       Publish any publishable assets from vendor packages

This simply updates the command to match - horizon:publish.

In addition it adjusts the order of commands in the service provider - the bottom half appeared to be alphabetical but the top half was not, so I made it all alphabetical.

@taylorotwell
Copy link
Member

I think this would technically need to go to master branch.

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

Successfully merging this pull request may close these issues.

2 participants