Skip to content
This repository has been archived by the owner on Oct 30, 2019. It is now read-only.

Fix misspellings. #133

Merged
merged 3 commits into from
Mar 7, 2018
Merged

Fix misspellings. #133

merged 3 commits into from
Mar 7, 2018

Conversation

Cabalist
Copy link
Contributor

@Cabalist Cabalist commented Mar 5, 2018

Description:

Ran a spellchecker against the code after noticing a few typos while installing.

Checklist:

  • [x ] The code change is tested and works locally.
  • [ N/A] Script has validation check of the job.

If pertinent:

  • [N/A ] Created/Updated documentation at /docs

@homeassistant
Copy link

Hi @Cabalist,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

make_package.sh Outdated
echo "Previous package exists. Deleting"
rm $PACKAGENAME
echo "Previous package exists. Deleting"
rm $PACKAGENAME

Choose a reason for hiding this comment

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

Double quote to prevent globbing and word splitting.

Landrash
Landrash previously approved these changes Mar 7, 2018
Copy link
Collaborator

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Thank you for the clean up! 🍪

Copy link
Collaborator

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

👍

@Landrash Landrash merged commit e1fc365 into home-assistant:dev Mar 7, 2018
@ludeeus ludeeus added this to the v0.8.0 milestone Mar 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants