Skip to content

Large Rework on Codebase#1929

Draft
s3inlc wants to merge 10 commits intodevfrom
testing
Draft

Large Rework on Codebase#1929
s3inlc wants to merge 10 commits intodevfrom
testing

Conversation

@s3inlc
Copy link
Member

@s3inlc s3inlc commented Feb 17, 2026

This PR brings various changes (mostly following the fact to better include the PHPUnit testing):

  • It allows to now run PHPUnit tests to internally test the backend (outside of the exposed API).
  • All classes now use namespacing according to the folder structure in the source code. This allows to get rid of most of the includes/requires (later all of them probably will be able to be removed).
  • Cleaned up the mess in use statements (especially use statements in middle or lower parts in code files and not on top)
  • API classes are now registered in the API index instead of the class files which gives more control when actually they are registered and allows better code inspection.
  • Applied some further typing.
  • All classes now live in their own file.

@s3inlc s3inlc requested a review from Copilot February 17, 2026 09:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@s3inlc s3inlc requested a review from Copilot February 17, 2026 15:13
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

1 participant