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

[GeoMechanicsApplication] Moved two #includes #12670

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Conversation

avdg81
Copy link
Contributor

@avdg81 avdg81 commented Sep 13, 2024

📝 Description

Attempt to include all interface dependencies in a header file to avoid that clients have to do that.

Attempt to include all interface dependencies in a header file to avoid that clients have to do that.
@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 Sep 13, 2024
@avdg81 avdg81 requested a review from WPK4FEM September 13, 2024 11:35
@avdg81 avdg81 self-assigned this Sep 13, 2024
@avdg81 avdg81 enabled auto-merge (squash) September 13, 2024 11:44
Copy link
Contributor

@WPK4FEM WPK4FEM left a comment

Choose a reason for hiding this comment

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

cstddef now available from IntegrationScheme i.s.o. the derived LobattoIntegrationScheme.

@avdg81 avdg81 merged commit eba546a into master Sep 13, 2024
9 of 11 checks passed
@avdg81 avdg81 deleted the geo/add-missing-include branch September 13, 2024 13:18
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
Development

Successfully merging this pull request may close these issues.

2 participants