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

Catch keyboard interrupts #60

Merged
merged 3 commits into from
Mar 10, 2020
Merged

Catch keyboard interrupts #60

merged 3 commits into from
Mar 10, 2020

Conversation

Badgie
Copy link
Owner

@Badgie Badgie commented Feb 10, 2020

Resolves #59

Copy link
Collaborator

@falkecarlsen falkecarlsen left a comment

Choose a reason for hiding this comment

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

Nice, leaves with cleaner output. Also something I use at times to check top lists.

@falkecarlsen falkecarlsen changed the base branch from master to dev March 6, 2020 19:44
@Badgie Badgie added this to the v1.2 milestone Mar 6, 2020
@Badgie Badgie merged commit 18acab4 into dev Mar 10, 2020
@Badgie Badgie deleted the keyb-interrupt branch March 10, 2020 14:22
Badgie added a commit that referenced this pull request Mar 11, 2020
* Rework config and save to playlist arg (#68)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* Update conf.py

Co-Authored-By: Falke Carlsen <[email protected]>

* update readme

Co-authored-by: Falke Carlsen <[email protected]>

* resolve conflicts

* resolve conflicts

* resolve conflicts

* Track feature printing (#75)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* allow printing track features

* Enhance tune validation (#74)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* enhance validity check of tunining

* add version flag (#61)

* remove whitespace error in custom schemes (#62)

* Single playlist (#65)

* use single playlist

* update readme

* Add tuning option to print (#67)

* add tuning option to print

* update readme to reflect install changes

* spelling

* Catch keyboard interrupts (#60)

* catch keyboard interrupts

* catch interrupt on new input prompts

* add flag for changing playback device (#77)

* notify when recommendation yields few different tracks (#78)

* fix minor things before release

Co-authored-by: Falke Carlsen <[email protected]>
Badgie added a commit that referenced this pull request Mar 22, 2020
* Version 1.2 (#80)

* Rework config and save to playlist arg (#68)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* Update conf.py

Co-Authored-By: Falke Carlsen <[email protected]>

* update readme

Co-authored-by: Falke Carlsen <[email protected]>

* resolve conflicts

* resolve conflicts

* resolve conflicts

* Track feature printing (#75)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* allow printing track features

* Enhance tune validation (#74)

* rework config file & make config-modifying arguments more uniform

* add custom config parser

* finalize new config structure

* clean up conf file and add some print output

* allow adding and removing currently playing track to input playlist

* update readme

* reformat some older functions

* minor bug fixes

* add and update comments

* update intro text in readme

* remove redundant non-positional parameter assignments in function calls

* enhance validity check of tunining

* add version flag (#61)

* remove whitespace error in custom schemes (#62)

* Single playlist (#65)

* use single playlist

* update readme

* Add tuning option to print (#67)

* add tuning option to print

* update readme to reflect install changes

* spelling

* Catch keyboard interrupts (#60)

* catch keyboard interrupts

* catch interrupt on new input prompts

* add flag for changing playback device (#77)

* notify when recommendation yields few different tracks (#78)

* fix minor things before release

Co-authored-by: Falke Carlsen <[email protected]>

* fix error with blacklist being generated incorrectly (#81)

* add logger class

* change info colour

* convert print messages

* implement switches

* add newline to log file print

* add shorthands of verbose/quiet

* add verbose and debugging prints

* use file only with log flag and on crash

* update readme

* add more comprehensive debugging msgs for requests

* add a few debug messages

* add more debugs

* add log level names for debugging purposes

* remove base64 debug

Co-authored-by: Falke Carlsen <[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.

Catch KeyboardInterrupt exceptions gracefully
2 participants