Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

save auth key #737

Merged
merged 2 commits into from
Aug 16, 2023
Merged

save auth key #737

merged 2 commits into from
Aug 16, 2023

Conversation

mkmer
Copy link
Contributor

@mkmer mkmer commented Jul 25, 2023

Description:

Small bit of cleanup on the blinksync application: add back save .json key, exit when username is blank or auth fails, lint/black minor changes

Related issue (if applicable): fixes #

Checklist:

  • Local tests with tox run successfully PR cannot be meged unless tests pass
  • Changes tested locally to ensure platform still works as intended
  • Tests added to verify new code works

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #737 (1779a73) into dev (50b395d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #737   +/-   ##
=======================================
  Coverage   99.64%   99.64%           
=======================================
  Files           8        8           
  Lines        1394     1394           
=======================================
  Hits         1389     1389           
  Misses          5        5           
Flag Coverage Δ
unittests 99.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@fronzbot fronzbot merged commit 9fa0b2e into fronzbot:dev Aug 16, 2023
@fronzbot fronzbot deleted the BlinkSync-cleanup branch August 16, 2023 16:07
@fronzbot fronzbot mentioned this pull request Aug 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants