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

Switched from PSR-0 to PSR-4 autoloading. Cleaned up composer.json. #13

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

7ochem
Copy link
Contributor

@7ochem 7ochem commented Nov 25, 2021

  • Switched from PSR-0 to PSR-4 autoloading

  • Changed tests namespace from ZichtTest\Util\ to Zicht\Util\Test\

  • Removed Zicht/Util/(Test/) directory depth: moved all code up directly into src/ and test/

  • Cleaned up composer.json

  • Needs merge of Setup PHP 8 #12 first

@7ochem 7ochem force-pushed the fix/switched-to-psr-4-autoloading branch 3 times, most recently from 3c426dc to 7b83e8f Compare November 25, 2021 15:07
.gitignore Outdated Show resolved Hide resolved
Comment on lines +16 to +19
### Changed
- Switched from PSR-0 to PSR-4 autoloading
- Changed tests namespace from ZichtTest\Util\ to Zicht\Util\Test\
- Removed Zicht/Util/(Test/) directory depth: moved all code up directly into src/ and test/
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, goede toevoeging voor de 2.0

- Switched from PSR-0 to PSR-4 autoloading
- Changed tests namespace from ZichtTest\Util\ to Zicht\Util\Test\
- Removed Zicht/Util/(Test/) directory depth: moved all code up directly into src/ and test/
- Cleaned up composer.json
@7ochem 7ochem force-pushed the fix/switched-to-psr-4-autoloading branch from 7b83e8f to fc05177 Compare November 25, 2021 15:43
@7ochem 7ochem merged commit 67b08ba into release/2.x Nov 25, 2021
@7ochem 7ochem deleted the fix/switched-to-psr-4-autoloading branch November 25, 2021 15:54
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.

2 participants