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.

Sign up for GitHub

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

Jump to bottom

Rafactor to PHP 7.4 #38

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Rafactor to PHP 7.4 #38

wants to merge 4 commits into from

Conversation

rdohms
Copy link
Owner

@rdohms rdohms commented May 23, 2020 •
edited
Loading

Time to modernize this app and clean up.

  • Bump to 7.4
  • Modernize Code
  • Switch to Laminas
  • Rename Zend class to Laminas (BC Break)
  • Add phpstan
  • Add DMS code style
  • Port to GH Actions

rdohms added 2 commits May 13, 2020 18:33
@rdohms
Update Deps and PHP to 7.4
05f699a 
Drop support for PHP 5.3 and anything below 7.4. And move all
dependencies to modern versions for a good bit of housecleaning.
@rdohms
Modernize and improve
c4a42c5 
Ran all inspections to bring code to 7.4 and improve some old patterns.
@rdohms rdohms added BC-Break This issue or pull request breaks backwards compatibility Refactoring Improvements that don't affect the current behaviour labels May 23, 2020
@rdohms rdohms added this to the v4.0.0 milestone May 23, 2020
rdohms added 2 commits May 23, 2020 14:49
@rdohms
Coding Standard bump
471b8e1
@rdohms
Apply coding standard
edca4f6 
Use doctrine's coding standard with a couple changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
BC-Break This issue or pull request breaks backwards compatibility Refactoring Improvements that don't affect the current behaviour
Projects
None yet
Milestone
v4.0.0
Development

Successfully merging this pull request may close these issues.
1 participant
@rdohms