Skip to content

Conversation

@OleksiiBulba
Copy link
Member

  • Added an exception message to ServiceNotRegisteredException exception;
  • Added a property serviceId to ServiceNotRegisteredException;
  • Added .gitattributes files to avoid unused files in vendor folder;

- Added an exception message to ServiceNotRegisteredException exception;
- Added a property `serviceId` to ServiceNotRegisteredException;
- Added .gitattributes files to avoid unused files in vendor folder;

Signed-off-by: Oleksii Bulba <oleksii_bulba@epam.com>
@OleksiiBulba OleksiiBulba requested a review from Asisyas January 8, 2023 13:35
@OleksiiBulba OleksiiBulba force-pushed the chore/enhancement branch 13 times, most recently from f9e989b to b7bf814 Compare January 8, 2023 17:04
- Normalized composer.json file;
- Added phpmd, phpcs and phpunit to dev dependencies in composer.json file;
- Added ci.yaml and static.yaml GitHub actions;
- Added composer.json conflict with micro/autowire<1.1
- Removed micro/autowire from required and added it to suggest section;

Signed-off-by: Oleksii Bulba <oleksii_bulba@epam.com>
Asisyas
Asisyas previously requested changes Jan 8, 2023
@OleksiiBulba OleksiiBulba requested a review from Asisyas January 8, 2023 21:24
@OleksiiBulba OleksiiBulba dismissed Asisyas’s stale review January 8, 2023 21:29

Changes applied

@OleksiiBulba OleksiiBulba force-pushed the chore/enhancement branch 2 times, most recently from 14a96f6 to 43376f1 Compare January 8, 2023 21:33
- Fixed tests after removing AutowiredContainer;
- Enhanced unit tests;
- Added arguments to decorated closure: actual decorated service and current container;
- Dropped support for php 8.1;
- PHP Code Sniffer fixes;
- Updated example in README.md;

Signed-off-by: Oleksii Bulba <oleksii_bulba@epam.com>
@Asisyas Asisyas merged commit 580f2c6 into master Jan 9, 2023
@OleksiiBulba OleksiiBulba deleted the chore/enhancement branch January 9, 2023 12:36
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.

3 participants