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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added waits for UDEV rules (copy of @philfifi's pull) #28

Merged
merged 8 commits into from Sep 15, 2018

Conversation

winterscar
Copy link
Contributor

I saw Phil's pull request was not seeing any action, so I thought I'd do it myself. 馃槉

I don't actually know how to run tests (never used pytest) so I don't know if they will pass. I have however written them and added documentation.

@winterscar
Copy link
Contributor Author

Only took me 5 tries to get the damn thing to build 馃槉

@rm-hull rm-hull self-requested a review September 4, 2018 20:22
@rm-hull
Copy link
Owner

rm-hull commented Sep 4, 2018

I'll have a look at the weekend if thats ok - give me a nudge if I haven't reviewed by end of Sunday 馃槵

@winterscar
Copy link
Contributor Author

Hi. I forgot to give you that nudge, so here it is 馃憠
馃槂

@rm-hull
Copy link
Owner

rm-hull commented Sep 16, 2018

This was rolled into v0.3.3, but I skipped sending that to PyPi, so it is published in v0.3.4

@winterscar
Copy link
Contributor Author

Awesome. Thanks for your work on this project :)

@philfifi
Copy link

Hello, thanks for making that action :-)
馃憤

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.

None yet

3 participants