[CLEANUP] Move PHPUnit configuration to Build/#1568
Merged
oliverklee merged 1 commit intoMyIntervals:mainfrom Apr 16, 2026
Merged
[CLEANUP] Move PHPUnit configuration to Build/#1568oliverklee merged 1 commit intoMyIntervals:mainfrom
Build/#1568oliverklee merged 1 commit intoMyIntervals:mainfrom
Conversation
AnReiter
changed the title
Build/
|
coverage: 72.927%. remained the same — AnReiter:phpunited into MyIntervals:main |
oliverklee
added
the
developer-specific
oliverklee
requested changes
Apr 16, 2026
AnReiter
force-pushed
the
phpunited
branch
2 times, most recently
from
51325e6 to
b1e0534
Compare
This comment was marked as outdated.
This comment was marked as outdated.
AnReiter
force-pushed
the
phpunited
branch
5 times, most recently
from
19f0d40 to
f674991
Compare
There was a problem hiding this comment.
Pull request overview
This PR relocates the PHPUnit configuration under Build/ and updates the Composer test scripts to explicitly use that config file.
Changes:
- Update Composer test scripts to run PHPUnit with
Build/phpunit/config.xml. - Adjust PHPUnit config paths to reference
src/andtests/correctly from the new config location. - Add
/Build/toexport-ignorerules (and add an IDE workspace file).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
composer.json |
Points test scripts at the relocated PHPUnit config file. |
Build/phpunit/config.xml |
Updates test suite + whitelist paths relative to the new location. |
.gitattributes |
Excludes Build/ from git archive exports. |
.idea/workspace.xml |
Adds JetBrains IDE workspace state (likely unintended). |
Files not reviewed (1)
- .idea/workspace.xml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
oliverklee
approved these changes
Apr 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.