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

Fix #1946, Correct various typos in docs, comments and error messages #1915

Merged
merged 4 commits into from
Sep 10, 2021

Conversation

thnkslprpt
Copy link
Contributor

@thnkslprpt thnkslprpt commented Aug 30, 2021

Describe the contribution
Fixed various minor typos in README docs, comments and a few debug/error/description message strings.

Testing performed
None (non-executable code).

Expected behavior changes
None. Code execution is unchanged/unaffected.

System(s) tested on
n/a

Additional context
n/a

Code contributions
n/a

@thnkslprpt thnkslprpt marked this pull request as draft September 1, 2021 14:23
@thnkslprpt thnkslprpt marked this pull request as ready for review September 1, 2021 16:32
@astrogeco astrogeco added the community Community contribution, YAY! label Sep 1, 2021
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@@ -359,7 +359,7 @@ Updates the doxygen documentation for this parameter, as it was incorrectly spec

Choose a reason for hiding this comment

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

Get rid of extra space on line 355, between the words on and inclusion in Adds documentation on inclusion presence of null terminators for length parameters..

README.md Show resolved Hide resolved
docs/cFE Application Developers Guide.md Show resolved Hide resolved
docs/cFE Application Developers Guide.md Show resolved Hide resolved
astrogeco added a commit to nasa/cFS that referenced this pull request Sep 10, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Sep 10, 2021
@astrogeco
Copy link
Contributor

astrogeco commented Sep 10, 2021

Merge conflicts resolved and tested in astrogeco@64a6a59

Merged in 64a6a59

@astrogeco astrogeco merged commit 74d5c28 into nasa:main Sep 10, 2021
astrogeco pushed a commit to astrogeco/cFE that referenced this pull request Sep 11, 2021
Also fix typos in EDS XML docs
astrogeco added a commit to astrogeco/cFE that referenced this pull request Sep 11, 2021
Fix nasa#1946, Correct typos in docs, comments and error messages.

Part of nasa/cFS#352
@astrogeco astrogeco changed the title Fixed various typos in docs, comments and error messages. Fix #1946, Correct various typos in docs, comments and error messages. Sep 11, 2021
@astrogeco astrogeco changed the title Fix #1946, Correct various typos in docs, comments and error messages. Fix #1946, Correct various typos in docs, comments and error messages Sep 11, 2021
@thnkslprpt thnkslprpt deleted the FIX-CFE-TYPOS branch September 11, 2021 07:28
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution, YAY!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typos in documentation, print statements, and comments
4 participants