-
Notifications
You must be signed in to change notification settings - Fork 849
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
Fix spelling mistakes #1586
Fix spelling mistakes #1586
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for these fixes :bouqet:
Before you merge, could you please quickly fix the remaining spelling mistakes in the codebase
/*--- Check feassbility for Streamwise Periodic flow ---*/ | ||
/*--- Check feasibility for Streamwise Periodic flow ---*/ | ||
if (Kind_Streamwise_Periodic != ENUM_STREAMWISE_PERIODIC::NONE) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol, who does these mistakes ... oh, wait 🤦
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request introduces 46 alerts when merging 233fd6b into c100612 - view on LGTM.com new alerts:
|
Proposed Changes
Give a brief overview of your contribution here in a few sentences.
Fixing some spelling mistakes.
Related Work
Resolve any issues (bug fix or feature request), note any related PRs, or mention interactions with the work of others, if any.
PR Checklist
Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.