Skip to content

Conversation

laravel-shift
Copy link
Owner

This pull request contains code changes and comments for code detected as incompatible or unrecommended by the Laravel Linter. You should review each comment and commit changes to the shift-6329 branch before merging.

If there were items you feel could have been detected, please send your feedback to shift@laravelshift.com or via Twitter.

@laravel-shift
Copy link
Owner Author

⚠ Your Laravel framework dependency is not set to the latest version in Composer. Unless you have a requirement for using a specific version of Laravel, you should set this to "laravel/framework": "5.6.*".

@laravel-shift
Copy link
Owner Author

ℹ Your Laravel version is less than the latest version (5.6). Use Laravel Shift to not only upgrade your project to the latest version of Laravel, but resolve most of the lint detected here.

@laravel-shift
Copy link
Owner Author

⚠ Your app/Http/Middleware/TrustProxies.php differs from the Laravel default. You should compare your app/Http/Middleware/TrustProxies.php and with the latest version and make any updates.

@laravel-shift
Copy link
Owner Author

❌ The following configuration files differ from the Laravel 5.6 defaults. If you are modifying configuration values directly, you should consider using an environment variable instead. You should compare your configuration files against the Laravel 5.6 configuration files and make any updates.

  • config/app.php
  • config/broadcasting.php
  • config/filesystems.php
  • config/hashing.php
  • config/logging.php
  • config/queue.php

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