Skip to content

Conversation

@alongosz
Copy link
Member

@alongosz alongosz commented Oct 23, 2025

Note

Effectively targets 4.6 branch (changed from main).

🎫 Issue n/a

Related PRs:

Description:

For now this is a preview of changes needed if run against ibexa/code-style#18. See also #1756 for changes needed by the current ibexa/code-style:2.2.x-dev (main) (ibexa/code-style@62c1a78).

Files changed with ibexa/code-style#18:

   1) src/lib/Tab/Dashboard/PagerLocationToDataMapper.php (fully_qualified_strict_types, ordered_imports)
   2) tests/bundle/ControllerArgumentResolver/ContentTreeChildrenQueryArgumentResolverTest.php (fully_qualified_strict_types)
   3) tests/lib/Form/DataTransformer/LanguageTransformerTest.php (fully_qualified_strict_types)
   4) tests/lib/Form/Data/FormMapper/ContentTypeDraftMapperTest.php (fully_qualified_strict_types, header_comment, blank_lines_before_namespace)
   5) tests/lib/Permission/LimitationResolverTest.php (fully_qualified_strict_types)

@alongosz alongosz changed the base branch from main to upgrade-to-code-style-2.2.x-dev October 26, 2025 21:32
@alongosz alongosz force-pushed the align-with-ibexa-code-style-18 branch 2 times, most recently from 62a2b99 to cc341ec Compare October 27, 2025 09:40
Files changed:

   1) src/lib/Tab/Dashboard/PagerLocationToDataMapper.php (fully_qualified_strict_types, ordered_imports)
   2) tests/bundle/ControllerArgumentResolver/ContentTreeChildrenQueryArgumentResolverTest.php (fully_qualified_strict_types)
   3) tests/lib/Form/DataTransformer/LanguageTransformerTest.php (fully_qualified_strict_types)
   4) tests/lib/Form/Data/FormMapper/ContentTypeDraftMapperTest.php (fully_qualified_strict_types, header_comment, blank_lines_before_namespace)
   5) tests/lib/Permission/LimitationResolverTest.php (fully_qualified_strict_types)
@alongosz alongosz force-pushed the align-with-ibexa-code-style-18 branch from cc341ec to 55b887c Compare October 27, 2025 13:08
@sonarqubecloud
Copy link

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.

4 participants