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 typos #17

Closed
wants to merge 1 commit into from
Closed

fix typos #17

wants to merge 1 commit into from

Conversation

spaette
Copy link

@spaette spaette commented Aug 8, 2024

a couple of typos

Copy link

github-actions bot commented Aug 8, 2024

Hello! The Git Commit Checker CI bot found a few problems with this PR:

ab01bb1: typos

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@jsquyres
Copy link
Member

jsquyres commented Aug 8, 2024

@spaette Thanks for the fixes!

Can you add the "Signed off by" to your commit message as per https://docs.open-mpi.org/en/v5.0.x/contributing.html#contributor-s-declaration?

@spaette
Copy link
Author

spaette commented Aug 8, 2024

my circumstance isn't favourable to add the "Signed off by"

this is a small fix, so feasibly a Member might implement it

grep output

$ grep -nr prehaps oac
oac/oac_init.m4:18:dnl Override (prehaps temporarily) the prefix name specified to
$ grep -nr seperated oac
oac/oac_var_scope.m4:71:dnl 1 -> space seperated list of variable names to push into the new scope
oac/oac_list.m4:27:dnl OAC_UNIQ: Uniqify the string-seperated words in the input variable
$ 

GNU sed commands

$ sed -i "s/prehaps/perhaps/g" oac/oac_init.m4
$ sed -i "s/seperated/separated/g" oac/oac_var_scope.m4
$ sed -i "s/seperated/separated/g" oac/oac_list.m4
$

@spaette spaette closed this Aug 8, 2024
@jsquyres
Copy link
Member

@spaette Got it; thank you!

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

Successfully merging this pull request may close these issues.

3 participants