Skip to content

Fixes #8141 #8195

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

Closed
wants to merge 2 commits into from
Closed

Conversation

kenmaca
Copy link
Contributor

@kenmaca kenmaca commented Mar 5, 2017

Validators for dashed linestyles now allow None as an allowed value along with floats if optional allow_none kwarg flag in validate_nseq_float.

Other validators that use validate_nseq_float aren't affected, so this bugfix won't have any breaking changes.

Validators for dashed linestyles now allow None as an allowed value along with floats through optional `allow_none` kwarg in validate_nseq_float. Other validators that use validate_nseq_float arent affected
@QuLogic
Copy link
Member

QuLogic commented Mar 5, 2017

Please give your PRs and commits descriptive titles; in 20 years no-one's going to know what "#8141" means.

@kenmaca kenmaca closed this Mar 5, 2017
@QuLogic
Copy link
Member

QuLogic commented Mar 5, 2017

Oh, sorry, you didn't need to open a new PR; there's an edit button on the top right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants