Add CI tests + config files - Githubissues

NiklasReiche / ad-spotify-mood-lights-sync

AppDaemon app that synchronizes rgb lights to the mood of the currently playing spotify song in Home Assistant.

MIT License

37 stars 5 forks source link

Add CI tests + config files #2

Closed ericmatte closed 3 years ago

ericmatte commented 3 years ago

Hi @NiklasReiche,

First of all, thank you for your interest in my ad-media-lights-sync project 😄

I didn't know that you could that easily test AppDaemon apps using appdaemontestframework. So now I've added a bunch of tests to my app which is great.

I just thought that I could share with you some of the thing I used in my flow.

Proposed changes with this PR

Update GitHub Actions flow to also run tests and linting on every push
Add autopep8 for auto-formatting and code style convention
Add requirements.txt with the list of all the required packages for contribution
Add build badge to README.md + auto format

Example run: https://github.com/ericmatte/ad-spotify-mood-lights-sync/actions/runs/591462931

Best regards,

Eric

NiklasReiche commented 3 years ago

Hey, thanks for sharing and implementing the changes 😄 This will especially be useful once I add some proper tests.

Everything looks good, so I'll just merge it now and then later do the workflow separation I mentioned above.