Skip to content

Turn --warn-no-return on by default #2462

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

Merged
merged 2 commits into from
Feb 7, 2017
Merged

Conversation

ddfisher
Copy link
Collaborator

--warn-no-return has shown itself to be generally useful, and so far has had only minor bugs (which have been fixed) over hundreds of thousands of lines of code. This sort of check is pretty integral to correctness -- let's turn it on by default.

@JukkaL
Copy link
Collaborator

JukkaL commented Nov 16, 2016

Can you fix the failing tests?

@ddfisher
Copy link
Collaborator Author

Working on it! Got busy with other tasks right after submitting this.

@JukkaL
Copy link
Collaborator

JukkaL commented Nov 21, 2016

This generates a large number of errors in the Dropbox 'C' repo. Are you planning to use -no-warn-no-return for it?

@ddfisher
Copy link
Collaborator Author

Thanks for double checking that! Yeah, I have a WIP diff to that repo to fix those -- the errors are legitimate.

@ddfisher ddfisher force-pushed the warn-no-return-default branch 2 times, most recently from 3d0df7f to cd2606e Compare February 3, 2017 01:38
@ddfisher ddfisher force-pushed the warn-no-return-default branch from cd2606e to 8242c36 Compare February 7, 2017 19:30
@ddfisher
Copy link
Collaborator Author

ddfisher commented Feb 7, 2017

@JukkaL this should be mergeable now!

@gvanrossum gvanrossum merged commit ae47a3e into master Feb 7, 2017
@gvanrossum gvanrossum deleted the warn-no-return-default branch February 7, 2017 20:24
@gvanrossum
Copy link
Member

Yay!

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