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 special handling for empty list to `none?/1` #2

Merged
merged 1 commit into from Nov 20, 2018

Conversation

Projects
None yet
3 participants
@janpieper
Contributor

janpieper commented Nov 20, 2018

This PR fixes issue #1.

@coveralls

This comment has been minimized.

coveralls commented Nov 20, 2018

Pull Request Test Coverage Report for Build 74

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.5%) to 97.5%

Totals Coverage Status
Change from base Build 72: 2.5%
Covered Lines: 39
Relevant Lines: 40

💛 - Coveralls
@Zeeker

Zeeker approved these changes Nov 20, 2018

@Zeeker Zeeker merged commit 4fe43bb into Zeeker:master Nov 20, 2018

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
coverage/coveralls Coverage increased (+2.5%) to 97.5%
Details
@Zeeker

This comment has been minimized.

Owner

Zeeker commented Nov 22, 2018

I've published brex 0.2.1 with the changes. Thanks for submitting. 💙💜❤️💛💚

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