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

Modernize code to php7.4 #6043

Open
wants to merge 30 commits into
base: edge
Choose a base branch
from

Conversation

ColonelMoutarde
Copy link
Contributor

Changes proposed in this pull request:

  • Modernize code to php7.4

Pull request checklist:

  • clear commit messages
  • code manually tested

@ColonelMoutarde ColonelMoutarde marked this pull request as draft January 16, 2024 20:27
@ColonelMoutarde ColonelMoutarde marked this pull request as ready for review January 16, 2024 21:28
@Alkarex
Copy link
Member

Alkarex commented Jan 17, 2024

Let's not hurry too much for those many but minor changes, as I would like to prioritise new features and merging as many as possible of the existing pending PRs, several of which have been waiting for a long time.
Merging such a PR will lead to many unneeded conflicts

@ColonelMoutarde ColonelMoutarde force-pushed the feature/modernize-code-to-php74 branch from d45487d to 0eb50f6 Compare March 7, 2024 16:38
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

2 participants