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

run test suite with PHP8 #1615

Merged
merged 4 commits into from
Nov 9, 2020
Merged

run test suite with PHP8 #1615

merged 4 commits into from
Nov 9, 2020

Conversation

Nielsvanpach
Copy link
Member

  • As discussed with @freekmurze , we can drop support on L5.8
  • dropped deprecated --no-suggest option in composer v2

@Nielsvanpach Nielsvanpach requested a review from drbyte November 9, 2020 14:32
Removed testbench 3.8 as well since we're also removing L5.8
@drbyte drbyte merged commit 2b8c2b1 into master Nov 9, 2020
@drbyte
Copy link
Collaborator

drbyte commented Nov 9, 2020

@Nielsvanpach
Thanks for this.

I'm leaning towards releasing this as a 4.0 for the package, so that the 3.x will always work for Laravel 5.8 sites.

Plus, a couple open PRs related to guards are best-suited for a major version release, so I'm inclined to merge them into that before tagging it.
(And due to a large project deadline this week, I may not get to all that until the weekend.)

So ... while I'm merging this into master, I'm not planning to tag a release for it just yet. People wanting to test this with their app can use dev-master.

@drbyte drbyte deleted the support-php8 branch November 9, 2020 16:51
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