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

[Core] Fix typos in kratos/processes subdir #10333

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

luzpaz
Copy link
Contributor

@luzpaz luzpaz commented Sep 30, 2022

📝 Description

Fix typos in kratos/processes subdir using codespell

🆕 Changelog

-Fixed doxygen and source comment typos in kratos/processes subdir

@luzpaz luzpaz requested a review from a team as a code owner September 30, 2022 15:23
@loumalouomega
Copy link
Member

Okay from my side, let's see @KratosMultiphysics/technical-committee

@loumalouomega loumalouomega changed the title Fix typos in kratos/processes subdir [Core] Fix typos in kratos/processes subdir Sep 30, 2022
rubenzorrilla
rubenzorrilla previously approved these changes Sep 30, 2022
Copy link
Member

@rubenzorrilla rubenzorrilla left a comment

Choose a reason for hiding this comment

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

Only corrects typos in documentation. Thanks.

@luzpaz luzpaz changed the title [Core] Fix typos in kratos/processes subdir [Core] Fix typos in kratos/processes subdir Nov 30, 2022
@luzpaz
Copy link
Contributor Author

luzpaz commented Oct 11, 2023

should i rebase ?

@loumalouomega
Copy link
Member

should i rebase ?

Fixed

roigcarlo
roigcarlo previously approved these changes Oct 16, 2023
@luzpaz
Copy link
Contributor Author

luzpaz commented Feb 6, 2025

Shall we try again ?

@rubenzorrilla
Copy link
Member

Shall we try again ?

OK. But we need to merge the master again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants