Skip to content

adora84 #1

Open
kalaaji wants to merge 5 commits intosimplonco:masterfrom
kalaaji:master
Open

adora84 #1
kalaaji wants to merge 5 commits intosimplonco:masterfrom
kalaaji:master

Conversation

@kalaaji
Copy link
Copy Markdown

@kalaaji kalaaji commented Jun 2, 2016

No description provided.

@yvan-sraka yvan-sraka changed the title adora test adora84 Jun 3, 2016
@yvan-sraka
Copy link
Copy Markdown
Contributor

yvan-sraka commented Jun 14, 2016

I check the Travis CI logs:
AssertionError: None is not True | False => I think your function return nothing..

Comment thread year.py
if n % 400 == 0:
print n, "it is a leap year."
elif n % 4 != 0:
print n, "it is not a leap year."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm right, you just print the result instead of return it 😉 !

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.

2 participants