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

[StructuralMechanicsApplication][ConstitutiveLawsApplication] Updating README #10896

Merged
merged 9 commits into from
Nov 18, 2024

Conversation

loumalouomega
Copy link
Member

📝 Description

Adding table to CL and structural app. @AlejandroCornejo and @SergioJimenezReyes please complete the CL README with the new laws

🆕 Changelog

@loumalouomega loumalouomega added Applications Documentation FastPR This Pr is simple and / or has been already tested and the revision should be fast labels Mar 20, 2023
@loumalouomega loumalouomega requested a review from a team as a code owner March 20, 2023 11:03
@loumalouomega
Copy link
Member Author

@rubenzorrilla check that the description I added for the new mixed elements is correct

Co-authored-by: Philipp Bucher <[email protected]>
@rubenzorrilla
Copy link
Member

@rubenzorrilla check that the description I added for the new mixed elements is correct

It is. Thanks!

Co-authored-by: Philipp Bucher <[email protected]>
@loumalouomega
Copy link
Member Author

Do we merge this?

@AlejandroCornejo
Copy link
Member

If we agree on doing this I want to add some more descriptions.. :) We will push this forward? @KratosMultiphysics/implementation-committee @KratosMultiphysics/technical-committee

@loumalouomega
Copy link
Member Author

@AlejandroCornejo should I close this or do you agree?

@AlejandroCornejo
Copy link
Member

@AlejandroCornejo should I close this or do you agree?

ok for me! I can add extra CLs later on!

@rubenzorrilla
Copy link
Member

Fine for me. It would be great to have this merged for the workshop.

@loumalouomega
Copy link
Member Author

Fine for me. It would be great to have this merged for the workshop.

Late. Can we merge this?

@loumalouomega loumalouomega merged commit bcc5801 into master Nov 18, 2024
11 checks passed
@loumalouomega loumalouomega deleted the structural/update-README-md branch November 18, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Applications Documentation FastPR This Pr is simple and / or has been already tested and the revision should be fast
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants