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

move HttpSecurityDsl and common files to annotation package #10474

Closed
wants to merge 2 commits into from

Conversation

nor-ek
Copy link
Contributor

@nor-ek nor-ek commented Nov 5, 2021

According to gh-10333 issue I've moved files from org.springframework.security.config.web.servlet to org.springframework.security.config.annotation.web in src/main/kotlin.

Result after running gradle check && gradle s101 | grep feedbackDependencies command

5-Nov-2021 11:53:32,153 [main] INFO - GREAT! feedbackDependencies has decreased by 28.0 to 0 (fail-on-feedback-dependencies)

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 5, 2021
@jzheaux jzheaux self-assigned this Nov 5, 2021
@jzheaux jzheaux added in: config An issue in spring-security-config type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 5, 2021
@jzheaux jzheaux added this to the 6.x milestone Nov 5, 2021
@jzheaux jzheaux added the status: duplicate A duplicate of another issue label Nov 5, 2021
Copy link
Contributor

@eleftherias eleftherias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @nor-ek!
I have left one comment inline.


import io.mockk.Called
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch finding these unused imports @nor-ek!
Since this change is not related to resolving package tangles, please move it to a separate commit.
This will ensure the commit message accurately describes the changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eleftherias please check it out. I've noticed more changes like that and packed them into Import cleanup commit.

I've also cleaned up titles and messages.

@jzheaux jzheaux added the type: breaks-passivity A change that breaks passivity with the previous release label Nov 5, 2021
@jzheaux jzheaux modified the milestones: 6.x, 6.0.0-M1 Nov 5, 2021
spring-projectsgh-10333 move dsl files from org.springframework.security.config.web.servlet to org.springframework.security.config.annotation.web in tests and src.
spring-projectsgh-10333 remove unnecessary imports
@jzheaux
Copy link
Contributor

jzheaux commented Nov 19, 2021

Thanks, @nor-ek! This was merged into main via 4f186f2.

Also, I re-established the package tangle baseline 4374905.

@jzheaux jzheaux closed this Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: config An issue in spring-security-config status: duplicate A duplicate of another issue 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.

None yet

4 participants