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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix handling of typing.Any #29

Merged
merged 2 commits into from
Apr 1, 2020
Merged

Conversation

erikseulean
Copy link
Contributor

So far Any was not taken in consideration and it was failing
to handle it correctly. If the base_type is Any, stop the
recursive check and return as if everything is fine.

Signed-off-by: Erik-Cristian Seulean [email protected]

*Issue number of the reported bug or feature request: #27

Copy link

@alexchamberlain alexchamberlain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there are several formatting changes that are unnecessary? Can we separate those out?

The actual change LGTM

@erikseulean
Copy link
Contributor Author

Yeah, I messed up something with the formatting, but I'll fix it later today.

@gaborbernat
Copy link
Contributor

@erikseulean if you allow another maintainer to push (checkbox on right) can help you out with this.

@erikseulean
Copy link
Contributor Author

It's checked.

@erikseulean
Copy link
Contributor Author

There's also a typo in the commit message that I'll have to fix

@gaborbernat
Copy link
Contributor

Hmm, it rejects me to push commits on top of it, maybe then I don't have maintainer rights here.

Signed-off-by: Erik-Cristian Seulean <[email protected]>
So far Any was not taken in consideration and it was failing
to handle it correctly. If the base_type is Any, stop the
recursive check and return as if everything is fine.

Signed-off-by: Erik-Cristian Seulean <[email protected]>
@erikseulean
Copy link
Contributor Author

I moved the formatting on a different commit.

@gaborbernat gaborbernat merged commit 56819ae into bloomberg:master Apr 1, 2020
keiclone pushed a commit to keiclone/attrs-strict that referenced this pull request May 26, 2020
* Black format

Signed-off-by: Erik-Cristian Seulean <[email protected]>

* Fix handling of typing.Any

So far Any was not taken in consideration and it was failing
to handle it correctly. If the base_type is Any, stop the
recursive check and return as if everything is fine.

Signed-off-by: Erik-Cristian Seulean <[email protected]>
Signed-off-by: Wilfred Wong <[email protected]>
keiclone pushed a commit to keiclone/attrs-strict that referenced this pull request May 26, 2020
* Black format

Signed-off-by: Erik-Cristian Seulean <[email protected]>

* Fix handling of typing.Any

So far Any was not taken in consideration and it was failing
to handle it correctly. If the base_type is Any, stop the
recursive check and return as if everything is fine.

Signed-off-by: Erik-Cristian Seulean <[email protected]>
@erikseulean erikseulean changed the title Fix handlying of typing.Any Fix handling of typing.Any Jun 30, 2020
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.

3 participants