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

chore: fix spelling #219

Merged
merged 3 commits into from
Mar 9, 2021
Merged

chore: fix spelling #219

merged 3 commits into from
Mar 9, 2021

Conversation

jbampton
Copy link
Member

@jbampton jbampton commented Mar 9, 2021

No description provided.

README.md Outdated
@@ -435,7 +435,7 @@ The following optional parameters are supported:
- `annotations` - object containing annotations for the action (default: `{}`)
- `limits` - object containing limits for the action (default: `{}`)
- `kind` - runtime environment parameter, ignored when `action` is an object (default: `nodejs:default`)
- `version` - set semantic version of the action. If parameter is empty when create new action openwisk generate 0.0.1 value when update an action increase the patch version.
- `version` - set semantic version of the action. If parameter is empty when create new action openwhisk generate 0.0.1 value when update an action increase the patch version.
Copy link
Member

Choose a reason for hiding this comment

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

Grammar needs more fixing here and casing for ow should be OpenWhisk.

Separately we should remove the reference to version. It is incorrect entirely.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@rabbah rabbah merged commit 10fb2ad into apache:master Mar 9, 2021
@rabbah
Copy link
Member

rabbah commented Mar 9, 2021

If you merge one in, I will come back stronger with 2 !!

or 2 dozen 😅 Thanks for all the PRs @jbampton.

@jbampton jbampton deleted the fix-spelling branch March 10, 2021 12:11
@jbampton
Copy link
Member Author

Thanks for merging them all in @rabbah 👍

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.

2 participants