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

Enable PHP 8 #9

Closed
wants to merge 4 commits into from
Closed

Enable PHP 8 #9

wants to merge 4 commits into from

Conversation

Synchro
Copy link
Contributor

@Synchro Synchro commented Dec 15, 2020

This PR increases the version range of supported PHP versions to include 8.0, and adds testing on 8.0. This also required allowing psalm version 4. All tests pass.

@coveralls
Copy link

coveralls commented Dec 15, 2020

Pull Request Test Coverage Report for Build 54

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 51: 0.0%
Covered Lines: 120
Relevant Lines: 120

💛 - Coveralls

.travis.yml Show resolved Hide resolved
composer.json Show resolved Hide resolved
composer.json Show resolved Hide resolved
@radarlog
Copy link
Contributor

radarlog commented Dec 21, 2020

Please change target branch to develop

@Synchro
Copy link
Contributor Author

Synchro commented Dec 21, 2020

I have rebased my changes on develop and pushed it to my fork, but I don't think I can make this PR switch branches. Is there a way?

@Synchro
Copy link
Contributor Author

Synchro commented Dec 21, 2020

I created a new PR on the develop branch in #10.

@Synchro Synchro closed this Dec 21, 2020
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.

None yet

3 participants