Skip to content

Conversation

ronodhirSoumik
Copy link
Contributor

@ronodhirSoumik ronodhirSoumik commented Jun 22, 2025

closes #17309

  • Squash and Sign Off after review

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 22, 2025
@jzheaux
Copy link
Contributor

jzheaux commented Jun 24, 2025

Thanks, @ronodhirSoumik. This looks great. When you squash, will you please ensure that each class that is removed is in its own commit? In this way, if one class removal causes a problem, we can easily revert it.

@jzheaux jzheaux self-assigned this Jun 24, 2025
@jzheaux jzheaux added in: core An issue in spring-security-core type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 24, 2025
@jzheaux jzheaux mentioned this pull request Jun 24, 2025
4 tasks
@jzheaux jzheaux added the type: breaks-passivity A change that breaks passivity with the previous release label Jun 24, 2025
@jzheaux jzheaux added this to the 7.0.x milestone Jun 24, 2025
@jzheaux jzheaux changed the title Removed deprecated classes Remove deprecated classes Jun 24, 2025
@jzheaux jzheaux changed the title Remove deprecated classes Remove deprecated classes moved to other packages Jun 24, 2025
@ronodhirSoumik
Copy link
Contributor Author

Thanks, @ronodhirSoumik. This looks great. When you squash, will you please ensure that each class that is removed is in its own commit? In this way, if one class removal causes a problem, we can easily revert it.

Got it. I will finalize this PR with four class-specific commits.

@ronodhirSoumik ronodhirSoumik force-pushed the feature/removed-deprecated-classes branch 2 times, most recently from edc92c4 to 1ef5107 Compare June 28, 2025 18:06
@ronodhirSoumik
Copy link
Contributor Author

@jzheaux requesting for final review

@jzheaux jzheaux modified the milestones: 7.0.x, 6.5.x, 7.0.0-M1 Jul 3, 2025
@jzheaux jzheaux force-pushed the feature/removed-deprecated-classes branch from 1ef5107 to b02a45a Compare July 3, 2025 18:32
@jzheaux jzheaux enabled auto-merge (rebase) July 3, 2025 18:32
@jzheaux
Copy link
Contributor

jzheaux commented Jul 3, 2025

Thanks, @ronodhirSoumik! I've updated the commit messages to contain the ticket they are addressing. This will merge into main once the build completes.

@jzheaux jzheaux merged commit 06bd81b into spring-projects:main Jul 3, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core An issue in spring-security-core type: breaks-passivity A change that breaks passivity with the previous release type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove classes from old locations
3 participants