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.

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

Replace string contatenations by using StringBuilder and RawParameterizedSqlStatement #5733

Closed
wants to merge 8 commits into from
Closed

Replace string contatenations by using StringBuilder and RawParameterizedSqlStatement #5733

wants to merge 8 commits into from

Conversation

MalloD12
Copy link
Contributor

Impact

  • Bug fix (non-breaking change which fixes expected existing functionality)
  • Enhancement/New feature (adds functionality without impacting existing logic)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Fixes #5546

Things to be aware of

Things to worry about

Additional Context

@MalloD12 MalloD12 self-assigned this Mar 25, 2024
@MalloD12 MalloD12 removed their assignment Apr 4, 2024
@filipelautert
Copy link
Collaborator

Hey @MalloD12 - isn't this PR included in #5812 ?

@MalloD12
Copy link
Contributor Author

Hey @MalloD12 - isn't this PR included in #5812 ?

Exactly, this is what I was talking about yesterday during our conversation. I think we can close this one if you agree.

Thanks,
Daniel.

@filipelautert
Copy link
Collaborator

filipelautert commented Apr 18, 2024 •
edited

@MalloD12 so I'm closing this one, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filipelautert filipelautert Awaiting requested review from filipelautert filipelautert is a code owner

Assignees
No one assigned
Labels
TypeBug
Projects
Liquibase Open Source
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix string concatenation code-scan issues
2 participants
@MalloD12 @filipelautert