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

feature: create command to upgrate filament to the latest version #274

Merged
merged 2 commits into from
Mar 23, 2021

Conversation

icarojobs
Copy link
Contributor

Command to upgrade Filament to the latest version

I thought about adding this contribution because instead of typing 5 commands to perform this task, we just typed 1.

@danharrin danharrin added the enhancement New feature or request label Mar 23, 2021
@danharrin
Copy link
Member

I like this! We won't need the alias though, the main command should be filament:upgrade.

{
system("composer update");

system("php artisan migrate");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we replace these calls with $this->call instead - it's definitely more reliable?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops only just seen this and I merged locally - will update.

@danharrin danharrin changed the base branch from main to develop March 23, 2021 10:35
@danharrin danharrin merged commit f06b3de into filamentphp:develop Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants