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

[php-di v7] DefinitionGlob #18

Open
wants to merge 9 commits into
base: mod7
Choose a base branch
from

Conversation

partikus
Copy link
Collaborator

@partikus partikus commented Jan 13, 2023

Version compatible with PHP ^8.0 as a successor of #2

Details (copied from #2)

DefinitionGlob to specify a pattern for definition files

  • there might be multiple definitions files scattered around in a modularized architecture
  • instead of listing all the files upfront, specify a glob pattern where to look for the files
  • directories would be scanned and files read lazily, when a definition is actually requested - to avoid performance hit on init

@partikus partikus force-pushed the feature/php-di-7-definition-glob branch from 57ce38a to 5dbcebe Compare January 13, 2023 15:56
@partikus partikus self-assigned this Jan 13, 2023
@partikus partikus force-pushed the feature/php-di-7-definition-glob branch 2 times, most recently from 7cf29d1 to ce2fca8 Compare January 10, 2024 07:47
@partikus partikus force-pushed the feature/php-di-7-definition-glob branch from ce2fca8 to 796ed99 Compare January 10, 2024 08:40
1. Detected deprecations in use:
- Rule "braces" is deprecated. Use "single_space_around_construct", "control_structure_braces", "control_structure_continuation_position", "declare_parentheses", "no_multiple_statements_per_line", "curly_braces_position", "statement_indentation" and "no_extra_blank_lines" instead.

2. Detected deprecations in use:
- Rule "curly_braces_position" is deprecated. Use "braces_position" instead.
@partikus partikus force-pushed the feature/php-di-7-definition-glob branch 2 times, most recently from 04d0ab7 to 772584d Compare January 10, 2024 12:55
mnapoli and others added 8 commits January 10, 2024 19:05
…ions

feat: php-cs-fixer - removing deprecations
- there might be multiple definitions files scattered around in a modularized architecture
- instead of listing all the files upfront, specify a glob pattern where to look for the files
- directories would be scanned and files read lazily, when a definition is actually requested - to avoid performance hit on init
@partikus partikus force-pushed the feature/php-di-7-definition-glob branch from 772584d to e6fdfde Compare January 11, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants