Skip to content

[WIP] Support for laravel 5.6 #52

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

Merged
merged 13 commits into from
Aug 8, 2018
Merged

Conversation

andrewmclagan
Copy link

@andrewmclagan andrewmclagan commented Jul 5, 2018

This is more a proposal to migrate some core changes and support php7 / larave 5.6 on a new 3.x.x tagging moving forward and older versions on the 2.x.x tagging:

  • Added direct support for laravel 5.6.x
  • Changed php support to min of >=7.1 as per laravel 5.6
  • Changed reqiured dependancy illuminate/support to minimum ^5.0
  • Added local docker phpunit, phpunit-versions, composer helpers ./bin/
    • e.g. simply type bin/phpunit
  • Updated travis config
  • Use of class constants to add coverage of namespace collision edge cases
  • Added config helper to DRY codebase
  • Added basic method return type declarations (strong typing is a win:win)
  • Dropping support for HHVM as PHPUnit has now dropped support as of 6.x.x

Andrew McLagan added 7 commits July 5, 2018 11:47
* Changed php support to min of >=7.1 as per laravel 5.6
* Changed reqiured dependancy `illuminate/support` to minimum ^5.0
* Added local docker `phpunit`, `phpunit-versions`, `composer` helpers `./bin/`
    - e.g. simply type `bin/phpunit`
* Updated travis config
* Use of class constants to add coverage of namespace collision edge cases
* Added config helper to DRY codebase
* Added basic method return type declarations (strong typing is a win:win)
@andrewmclagan
Copy link
Author

See readme for some small docker changes

@ArturMoczulski ArturMoczulski self-assigned this Jul 16, 2018
@ArturMoczulski
Copy link

@andrewmclagan great PR. Thank you for your work on this. I will definitely use it as soon as I figure out the best way to fit this into our tagging policy.

@ArturMoczulski ArturMoczulski merged commit 98f82a3 into rollbar:master Aug 8, 2018
@ArturMoczulski
Copy link

This has been merged into master which will now be the Laravel 5.6 support line along with 3.x.x releases. We will still maintain support for Laravel 5.5 and below in the laravel-5.5 branch and in the 2.x.x releases.

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