Skip to content

Polish Pattern Matching Usage #16493

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 26, 2025
Merged

Polish Pattern Matching Usage #16493

merged 1 commit into from
Mar 26, 2025

Conversation

ngocnhan-tran1996
Copy link
Contributor

  • Use pattern matching
  • Simply if condition, parameterToken was checked null before, so we just check parameterToken is blank or not instead rechecking null

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 28, 2025
@ngocnhan-tran1996
Polish
3269017 
Signed-off-by: Tran Ngoc Nhan <ngocnhan.tran1996@gmail.com>
@jzheaux jzheaux self-assigned this Mar 26, 2025
@jzheaux jzheaux added type: enhancement A general enhancement in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 26, 2025
@jzheaux jzheaux added this to the 6.5.0-RC1 milestone Mar 26, 2025
@jzheaux jzheaux merged commit 7bca17c into spring-projects:main Mar 26, 2025
6 checks passed
@jzheaux
Copy link
Contributor

jzheaux commented Mar 26, 2025

Thanks for another polish, @ngocnhan-tran1996! This is now merged into main.

@jzheaux jzheaux changed the title Polish Polish Pattern Matching Usage Mar 26, 2025
@ngocnhan-tran1996 ngocnhan-tran1996 deleted the polish branch March 27, 2025 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jzheaux jzheaux

Labels
in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: enhancement A general enhancement
Projects
None yet
Milestone
6.5.0-RC1
Development

Successfully merging this pull request may close these issues.
3 participants
@ngocnhan-tran1996 @jzheaux @spring-projects-issues