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

Spelling fixes #1484

Merged
merged 1 commit into from
Jul 28, 2020
Merged

Spelling fixes #1484

merged 1 commit into from
Jul 28, 2020

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Jul 28, 2020

Pull in the latest rake task, fix some variable names, fix a few legit typos.

Signed-off-by: Tim Smith [email protected]

Pull in the latest rake task, fix some variable names, fix a few legit typos.

Signed-off-by: Tim Smith <[email protected]>
@tas50 tas50 requested review from a team as code owners July 28, 2020 17:15
@@ -56,7 +56,7 @@ def get_vm_attributes(vmtools_path)
end
end
# vmware-toolbox-cmd <param> status commands
# Iterate through each parameter supported by the "vwware-toolbox-cmd status" command, assign value
# Iterate through each parameter supported by the "vmware-toolbox-cmd status" command, assign value
Copy link
Contributor

Choose a reason for hiding this comment

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

I hope Volkswagen has software somewhere called "VWware".

@@ -17,17 +17,31 @@

namespace :spellcheck do
task :fetch_common do
sh "wget https://raw.githubusercontent.com/chef/chef_dictionary/master/chef.txt -O chef_dictionary.txt"
sh "wget -q https://raw.githubusercontent.com/chef/chef_dictionary/master/chef.txt -O chef_dictionary.txt"
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this file need to go in .gitignore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's already in there

@tas50 tas50 merged commit 95e033e into master Jul 28, 2020
@tas50 tas50 deleted the spelling branch October 30, 2020 22:16
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