Skip to content

Conversation

@peteretep
Copy link

No description provided.

@jane-t
Copy link

jane-t commented Apr 13, 2021

Line 203 needs changing to
if args.username and args.password

@peteretep
Copy link
Author

oh yes, of course. thank you.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 744639118

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.915%

Totals Coverage Status
Change from base Build 363426739: 0.0%
Covered Lines: 112
Relevant Lines: 118

💛 - Coveralls

@drkpxl
Copy link

drkpxl commented May 12, 2021

This is awesome. Thank you. I was pulling my hair out trying to figure out syntax! Please merge this into master!

@Gadgetoid Gadgetoid merged commit ae72dab into pimoroni:master Aug 31, 2021
@Gadgetoid
Copy link
Member

This is awesome. Thank you. I was pulling my hair out trying to figure out syntax! Please merge this into master!

I'm a day late and a dollar short. But thank you all, and done!

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.

6 participants