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

Add missing return statement. #4273

Merged
merged 1 commit into from Jul 17, 2016
Merged

Add missing return statement. #4273

merged 1 commit into from Jul 17, 2016

Conversation

aaugustin
Copy link
Contributor

@aaugustin aaugustin commented Jul 16, 2016

Fix #4272.

@aaugustin
Copy link
Contributor Author

aaugustin commented Jul 16, 2016

I talked quickly with @xordoquy. He wanted to look into adding a test.

@codecov-io
Copy link

codecov-io commented Jul 16, 2016

Current coverage is 91.18%

Merging #4273 into master will not change coverage

@@             master      #4273   diff @@
==========================================
  Files            52         52          
  Lines          5776       5776          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           5267       5267          
  Misses          509        509          
  Partials          0          0          

Powered by Codecov. Last updated by 91aad92...a436515

@xordoquy
Copy link
Collaborator

xordoquy commented Jul 17, 2016

Opened an issue about the test so we can move forward with this.

@xordoquy xordoquy merged commit 6b71320 into encode:master Jul 17, 2016
@xordoquy
Copy link
Collaborator

xordoquy commented Jul 17, 2016

Thanks for all the work !

@tomchristie
Copy link
Member

tomchristie commented Jul 18, 2016

Thanks @aaugustin - This'll be worth a new minor release shortly.

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

Successfully merging this pull request may close these issues.

None yet

5 participants