search

hamcrest / PyHamcrest

Hamcrest matchers for Python
http://hamcrest.org/
Other
773 stars 112 forks source link

Don't recommend "import *" form. #197

Closed brunns closed 2 years ago

offbyone commented 2 years ago

Okay, I won't merge this super hastily; if you need to add more, just merge 'em when you're done :)

brunns commented 2 years ago

Yeah, sorry - I was being lazy, editing through the web. I'll do it properly...

(I need to look at the build, too.)

offbyone commented 2 years ago

Yeah, that's on my list too; bit rot... yay.

brunns commented 2 years ago

I've been considering running my CI builds on a schedule as well as on push, like so, so I'd at least get an early heads up to stuff breaking due to something 3rd party.

brunns commented 2 years ago

Would you consider something like https://github.com/brunns/mbtest/blob/master/.github/workflows/ci.yml#L6?

offbyone commented 2 years ago

Yeah, definitely.