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

Apply fixes from StyleCI #251

Merged
merged 1 commit into from
Aug 25, 2017
Merged

Conversation

kattrali
Copy link
Contributor

This pull request applies code style fixes from an analysis carried out by StyleCI.


For more information, click here.

[ci skip] [skip ci]
@kattrali kattrali merged commit 3217e5d into cawllec/notify-console-commands Aug 25, 2017
@kattrali kattrali deleted the analysis-XV2Eyg branch August 25, 2017 16:28
Cawllec pushed a commit that referenced this pull request Sep 29, 2017
Cawllec added a commit that referenced this pull request Oct 3, 2017
* Added ConsoleRequest to resolver to attach console metadata

* Now using globally available PHP builtin

* Updated test to reflect console update

* Apply fixes from StyleCI (#247)

* Using laravel specific objects

* Apply fixes from StyleCI (#249)

* No longer user globals

* Apply fixes from StyleCI (#250)

* Added new console request class usage

* Updated test

* Apply fixes from StyleCI (#251)

* Bumped required bugsnag version

* Added ConsoleRequest to resolver to attach console metadata

* Now using globally available PHP builtin

* Updated test to reflect console update

* Apply fixes from StyleCI (#247)

* Using laravel specific objects

* Apply fixes from StyleCI (#249)

* No longer user globals

* Apply fixes from StyleCI (#250)

* Added new console request class usage

* Updated test

* Apply fixes from StyleCI (#251)

* Bumped required bugsnag version

* Added safer command checking when retreiving console args

* Apply fixes from StyleCI (#256)
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.

1 participant