Skip to content

Mark AliasForSecurityEncoderCompilerPass as internal #7747

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

Merged
merged 5 commits into from
Jul 7, 2025

Conversation

mamazu
Copy link
Contributor

@mamazu mamazu commented Jan 21, 2025

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Fixed tickets -
Related issues/PRs #7743
License MIT
Documentation PR -

What's in this PR?

Mark the class as internal.

Why?

We need to remove it in 3.0

@mamazu mamazu changed the title Mark compiler pass as internal Mark AliasForSecurityEncoderCompilerPass as internal Jan 21, 2025
@mamazu mamazu added the DX Affecting the end developer label Jan 28, 2025
@mamazu mamazu force-pushed the internal_compilerpass branch from ae55b63 to a19962e Compare January 28, 2025 13:16
@mamazu mamazu added the Technical Debt Impacts code quality, no or just small impact on end developers and users label Jan 28, 2025
@mamazu
Copy link
Contributor Author

mamazu commented Feb 5, 2025

@alexander-schranz We probably don't need a pipeline for that. It's just a comment.

@mamazu
Copy link
Contributor Author

mamazu commented Jul 2, 2025

@alexander-schranz I think this one should also be easy to merge.

@alexander-schranz alexander-schranz merged commit a253f47 into sulu:2.6 Jul 7, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX Affecting the end developer Technical Debt Impacts code quality, no or just small impact on end developers and users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants