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

Added log functionality and easy hastebin sharing. #78

Merged
merged 7 commits into from
Jan 28, 2018
Merged

Added log functionality and easy hastebin sharing. #78

merged 7 commits into from
Jan 28, 2018

Conversation

ludeeus
Copy link
Member

@ludeeus ludeeus commented Dec 30, 2017

Added writing to an logfile for every install/upgrade, this "function" will overwrite the file so only the latest one is available.

Added "log" as an cmd option.
This will produce an hastebin link to easy share the log of the last install/upgrade.

@ludeeus ludeeus changed the title Logs Added log functionality and easy hastebin sharing Jan 6, 2018
@ludeeus
Copy link
Member Author

ludeeus commented Jan 6, 2018

If this is included, the scripts (that have verification check) can be edited to post an hastebin link when script fails.

@Landrash
Copy link
Collaborator

Landrash commented Jan 6, 2018

It might be wise to have a log only do a more on the file a create a share-log file to hastebin. Looks good else :)

@ludeeus
Copy link
Member Author

ludeeus commented Jan 6, 2018

Maybe, I did not know about more until now :P

@ludeeus ludeeus changed the title Added log functionality and easy hastebin sharing Added log functionality and easy hastebin sharing. Jan 12, 2018
@Landrash Landrash added this to the v0.7 milestone Jan 27, 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.

Looks good and tagged for 0.7.
A few conflicts between your pr's. Mind resolving them?

@ludeeus
Copy link
Member Author

ludeeus commented Jan 27, 2018

This should also be ok bow :)

@Landrash Landrash merged commit 7a60176 into home-assistant:dev Jan 28, 2018
@ludeeus ludeeus deleted the patch-2 branch January 28, 2018 12:47
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.

3 participants