Skip to content

Use repl package and remove mobiledetect#829

Merged
othercorey merged 1 commit into4.nextfrom
use-repl
Mar 15, 2021
Merged

Use repl package and remove mobiledetect#829
othercorey merged 1 commit into4.nextfrom
use-repl

Conversation

@markstory
Copy link
Copy Markdown
Member

Instead of shipping the console command with the application skeleton using a separate package will allow us to deliver fixes to the console command in a reasonable way. It also enables us to provide more shortcut functions for use in the repl and development mode.

I've removed mobiledetect as a direct dependency. I don't think it gets a ton of usage by users and it can live on as a comment and optional package that can be installed as necessary.

Comment thread src/Application.php Outdated
Comment thread src/Application.php Outdated
@ADmad
Copy link
Copy Markdown
Member

ADmad commented Mar 14, 2021

I don't think it gets a ton of usage by users...

A lot of those developing CMSs would need it, probably more than the repl feature :)

@markstory
Copy link
Copy Markdown
Member Author

@ADmad Fair point I've put mobiledetect back.

Comment thread config/bootstrap.php Outdated
Comment thread src/Application.php Outdated
Comment thread src/Application.php Outdated
Instead of shipping the console command with the application skeleton
using a separate package will allow us to deliver fixes to the console
command in a reasonable way. It also enables us to provide more shortcut
functions for use in the repl and development mode.
@othercorey othercorey added this to the 4.next milestone Mar 15, 2021
@othercorey othercorey merged commit 36cb14f into 4.next Mar 15, 2021
@othercorey othercorey deleted the use-repl branch March 15, 2021 17:37
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.

6 participants