Skip to content

Fix PHP 8.2 compatibility#56

Merged
dadolun95 merged 1 commit intomainfrom
fix/php82-compatibility
Mar 20, 2026
Merged

Fix PHP 8.2 compatibility#56
dadolun95 merged 1 commit intomainfrom
fix/php82-compatibility

Conversation

@SamueleMartini
Copy link
Copy Markdown
Collaborator

@SamueleMartini SamueleMartini commented Mar 20, 2026 •
edited
Loading

Closes #55

Summary

  • Lowered minimum PHP version from 8.3 to 8.2 in composer.json
  • Removed typed constants (PHP 8.3 feature) from all classes for 8.2 compatibility
  • Updated CHANGELOG with 2.0.1, 2.0.2 and 2.1.0 entries

Test plan

  • Verify module installs on PHP 8.2 without errors
  • Verify module still works on PHP 8.3
  • Run No syntax errors detected in Standard input code on all module files with PHP 8.2

@SamueleMartini SamueleMartini requested review from a team and dadolun95 as code owners March 20, 2026 09:09
@SamueleMartini SamueleMartini mentioned this pull request Mar 20, 2026
Closed
dadolun95
dadolun95 approved these changes Mar 20, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@dadolun95 dadolun95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@dadolun95 dadolun95 merged commit a3b42e6 into main Mar 20, 2026
2 checks passed
@dadolun95 dadolun95 deleted the fix/php82-compatibility branch March 20, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dadolun95 dadolun95 dadolun95 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Module blocks installation of Mage-OS on PHP 8.2

2 participants

@SamueleMartini @dadolun95