question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Get code ready for HA inclusion

See original GitHub issue

Start reformatting files for HA Style guide. This is a requirement for HA submission

from what I can see

We can

  • issue black against the files
  • Check PEP8 using pycharm
  • run flake8 against our source.

_LOGGING comments should use the new %s style e…g _LOGGER.info("Can't connect to the webservice %s at %s", string1, string2) Latter looks like a recommendation so I may skip it

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Reactions:1
  • Comments:25 (16 by maintainers)

github_iconTop GitHub Comments

1reaction
msp1974commented, Jun 22, 2020

@asantaga hopefully you will see that I put a bit of time into this tonight to justify my collaborator status! 😉

Have addressed most of the items raised by Frenck (except the reduce capability points). Also did some learning on tox testing and amended the tests so that they should work fine now. Passed on my machine.

I’ll let you go back to Frenck re not reducing functionality.

KR Mark

0reactions
asantagacommented, Jul 15, 2020

Closing off, probably not going to do this

Read more comments on GitHub >

github_iconTop Results From Across the Web

Accessing CodeReady Containers on a Remote Server
CodeReady Containers, or CRC for short, is perfect for that. Rather than try to rephrase what it is, I'll just copy it directly...
Read more >
Small Business Training | About Verizon
Verizon Small Business Digital Ready. Free online curriculum with courses, expert coaching, peer networking and exclusive incentives and grant opportunities.
Read more >
HackerRank
HackerRank is the market-leading technical assessment and remote interview solution for hiring developers. Learn how to hire technical talent from anywhere!
Read more >
Z-Wave - Home Assistant
This integration allows you to control a Z-Wave network via the Z-Wave JS driver. This is our recommended Z-Wave integration for Home Assistant....
Read more >
Center for Inclusive Design and Engineering - CU Denver
Need Assistance? Find Us · Specialty AT Clinic · Colorado Tech Act Program · Preparing for Emergencies · Funding and Finding AT Devices....
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found