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

Periodic spelling sweep #408

Merged
merged 5 commits into from
May 23, 2019
Merged

Periodic spelling sweep #408

merged 5 commits into from
May 23, 2019

Conversation

msorens
Copy link
Contributor

@msorens msorens commented May 23, 2019

🔩 Description

This is a periodic sweep of spelling corrections and exceptions across Automate.
As there are a few new components there is a larger swath of files in this PR.
This is generated by running ./scripts/spellcheck from the automate repo root.

(Eventually the spell checker will be wired up to the build itself, but have not gotten to that yet.)

👍 Definition of Done

Almost all spelling errors resolved.

👟 Demo Script / Repro Steps

⛓️ Related Resources

✅ Checklist

  • Necessary tests added/updated?
  • Necessary docs added/updated?
  • Code actually executed?
  • Vetting performed (unit tests, lint, etc.)?

@msorens msorens self-assigned this May 23, 2019
Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -181,7 +181,7 @@ handle_metadata({ok, IdPMetadata}, SSOBinding, NameId) ->
handle_return([], _) -> {error, not_found};
handle_return([Config], EntName) -> {ok, deserialize(Config, EntName)};
handle_return({error, Why}, _) ->
chef_log:error("An internal server error occured in trying to perform a db operation on SAML configuration. ~n~p", [Why]),
chef_log:error("An internal server error occurred in trying to perform a db operation on SAML configuration. ~n~p", [Why]),
Copy link
Contributor

Choose a reason for hiding this comment

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

how dare you

msorens added 5 commits May 23, 2019 10:27
Signed-off-by: michael sorens <[email protected]>
Signed-off-by: michael sorens <[email protected]>
Signed-off-by: michael sorens <[email protected]>
Signed-off-by: michael sorens <[email protected]>
Signed-off-by: michael sorens <[email protected]>
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