Skip to content

[GeoMechanicsApplication] Removed three empty statements#12390

Merged
avdg81 merged 1 commit intomasterfrom
geo/remove-empty-statements
May 21, 2024
Merged

[GeoMechanicsApplication] Removed three empty statements#12390
avdg81 merged 1 commit intomasterfrom
geo/remove-empty-statements

Conversation

@avdg81
Copy link
Contributor

@avdg81 avdg81 commented May 21, 2024

📝 Description
Removed three semicolons from a unit test file that were accidentally appended to check macros.

@avdg81 avdg81 added Cleanup FastPR This Pr is simple and / or has been already tested and the revision should be fast GeoMechanics Issues related to the GeoMechanicsApplication labels May 21, 2024
@avdg81 avdg81 requested a review from rfaasse May 21, 2024 07:28
@avdg81 avdg81 self-assigned this May 21, 2024
Copy link
Contributor

@rfaasse rfaasse left a comment

Choose a reason for hiding this comment

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

Simple, but removes code smells, thanks for being on top of this!

@avdg81 avdg81 merged commit 8419608 into master May 21, 2024
@avdg81 avdg81 deleted the geo/remove-empty-statements branch May 21, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cleanup FastPR This Pr is simple and / or has been already tested and the revision should be fast GeoMechanics Issues related to the GeoMechanicsApplication

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants