-
-
Notifications
You must be signed in to change notification settings - Fork 374
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding matches_re validator #552
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know you said you're not done yet but here's some feedback so you can avoid detours when you'll find the time.
Also a changelog entry please. :)
thanks for the comments, will address soon sorry for the slowness, I've been a bit hung up on getting the local dev environment set up, I keep running into this:
|
I wasn’t aware of that problem until recently. Could you help my understand how it happens in #561? |
I believe pip install -dev on windows should work now? |
yes! worked like a charm after update, thanks! |
just an FYI, pre-commit still fails with an SSL failure trying to fetch black from github Errors: I've checked the code is flake8 clean, hopefully I'm not missing anything else format wise |
Hm, that must be some local problem. Under what Python version is your pre-commit running? It must be something ancient, if the ssl is too old to connect to GitHub?! Anthony is AFAIK using Windows too, so that shouldn't be the issue. The tox env uses explicitly 3.7 tho? (I've took this opportunity to fix black's URL tho)
|
Hey a release is imminent; do you think you can bang it into shape on short notice or do you want to push this to the next release? |
let me give it one more push this weekend and see if I can resolve everything -- sorry for being so flakey! |
alright I think this is ready to go now ... only thing is I'm not sure about hypothesis or the type testing area not sure if those apply since this is more of a "leaf" than structural |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
still needs more work just getting the ball rolling
Pull Request Check List
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes. But please read our contribution guide at least once, it will save you unnecessary review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.
.pyi
).tests/typing_example.py
.docs/api.rst
by hand.@attr.s()
have to be added by hand too.versionadded
,versionchanged
, ordeprecated
directives..rst
files is written using semantic newlines.changelog.d
.If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!