Fix typos #2816
Fix typos #2816
Conversation
|
Change the base to Impressive number of PR's you've made to a few thousand repositories with typo fixes. Also please describe your methodology and use of AI to enhance the process. Before we can merge this, please use our pull request template in your description of the PR. We use that to weed out bot generated PRs. |
There was a problem hiding this comment.
Pull Request Overview
This pull request fixes various typos across multiple Fortran source files, C++ header files, and documentation. The changes primarily address spelling errors in comments, error messages, and documentation strings.
- Corrected various misspellings such as “occured” → “occurred”, “writting” → “writing”, “overwritting” → “overwriting”, “lenght” → “length”, “libary” → “library”, “intialization” → “initialization”, and duplicate words.
- Consistent spelling improvements throughout production code and documentation.
Reviewed Changes
Copilot reviewed 76 out of 76 changed files in this pull request and generated 19 comments.
Show a summary per file
| File | Description |
|---|---|
| modules/aerodyn/src/FVW_Subs.f90 | Fixed typo in error message text. |
| modules/aerodyn/src/FVW.f90 | Corrected typos in logging messages regarding VTK outputs. |
| modules/aerodyn/src/DBEMT.f90 | Fixed spelling mistake in comment text. |
| modules/aerodyn/src/AirfoilInfo.f90 | Corrected spelling of “overwriting” in comments. |
| modules/aerodyn/src/AeroDyn_Inflow_C_Binding.f90 | Corrected “lenght” to “length” and removed duplicate “the”. |
| modules/aerodyn/src/AeroDyn_Driver.f90 | Fixed “intialization” typo in user timing comments. |
| glue-codes/openfast-cpp/src/OpenFAST.H | Removed duplicated “the” in the comment for azimuth blending. |
| glue-codes/fast-farm/src/FAST_Farm_Subs.f90 | Corrected “libary” to “library” in a comment. |
| glue-codes/fast-farm/src/FAST_Farm.f90 | Corrected “intialization” to “initialization” in timing comments. |
| docs/source/user/subdyn/theory.rst | Fixed “prefered” to “preferred” in a documentation note. |
| docs/source/user/fast.farm/OutputFiles.rst | Removed duplicated “the” from text. |
| docs/source/user/fast.farm/FFarmTheory.rst | Corrected “paramters” to “parameters” in the theory description. |
| docs/source/user/elastodyn/theory.rst | Removed duplicated “the” in the documentation commentary. |
| docs/source/user/cppapi/files/cDriver.i | Removed duplicated “the” in configuration comments. |
| docs/source/user/beamdyn/theory.rst | Removed duplicated “the” in the blade dynamics description. |
| docs/source/user/aerodyn/theory.rst | Fixed “occured” to “occurred” in the introductory text. |
| docs/source/user/aerodyn/input.rst | Corrected “accross” to “across” in the input description. |
| docs/source/install/index.rst | Fixed “Defualt” to “Default” in a CMake option description. |
| docs/source/dev/index.rst | Corrected “prefered” to “preferred” in developer communication guidelines. |
Comments suppressed due to low confidence (6)
modules/aerodyn/src/AeroDyn_Inflow_C_Binding.f90:421
- Corrected typo in comment: changed 'lenght' to 'length'.
integer(c_int), intent(in) :: ADinputFileStringLength_C !< lenght of the input file string
modules/aerodyn/src/AeroDyn_Inflow_C_Binding.f90:424
- Corrected typo in comment: changed 'lenght' to 'length'.
integer(c_int), intent(in) :: IfWinputFileStringLength_C !< lenght of the input file string
modules/aerodyn/src/AeroDyn_Inflow_C_Binding.f90:2107
- Removed duplicate 'the' in the comment.
!! NOTE: the OriginInit is not included in the data passed in and must be added to the the position info here
modules/aerodyn/src/AeroDyn_Driver.f90:29
- Corrected typo: changed 'intialization' to 'initialization' in the comment.
REAL(ReKi) :: UsrTime2 ! User CPU time for simulation (without initialization) [(s)]
glue-codes/fast-farm/src/FAST_Farm.f90:51
- Corrected typo: changed 'intialization' to 'initialization' in the comment.
REAL(ReKi) :: SimStrtCPU !< User CPU time for simulation (without initialization)
glue-codes/fast-farm/src/FAST_Farm.f90:52
- Corrected typo: changed 'intialization' to 'initialization' in the comment.
REAL(ReKi) :: ProgStrtCPU !< User CPU time for program (with initialization)
| if (istat/=0) then | ||
| errStat2=ErrID_Fatal | ||
| errMsg2='Error occured while reading line '//num2lstr(i+1)//' of circulation file: '//trim(CirculationFileName) | ||
| errMsg2='Error occurred while reading line '//num2lstr(i+1)//' of circulation file: '//trim(CirculationFileName) |
There was a problem hiding this comment.
Corrected typo in error message: changed 'occured' to 'occurred'.
andrew-platt
left a comment
andrew-platt
left a comment
There was a problem hiding this comment.
Thanks for fixing those!
I recognize a few from 15+ years ago.
2 participants
Feature or improvement description
Fix typos in docs and comments
Related issue, if one exists
Impacted areas of the software
Additional supporting information
Test results, if applicable