Skip to content

Updated power_of_4.py #11152

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

Closed
wants to merge 2 commits into from
Closed

Conversation

AseemGupta39
Copy link

No description provided.

@algorithms-keeper algorithms-keeper bot added the tests are failing Do not merge until tests pass label Nov 11, 2023
Copy link
Contributor

@imSanko imSanko left a comment

Choose a reason for hiding this comment

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

Check with the pre-commit run --all-files --show-diff-on-failure for your errors to go through

return c % 2 == 1
else:
return False
while number != 1 and number != 2:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
while number != 1 and number != 2:
while number not in (1, 2):

Comment on lines -60 to -61
else:
return False
Copy link
Contributor

Choose a reason for hiding this comment

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

The function now doesn't return anything if the if-statement evaluates to False. This is why the build is failing.

@cclauss
Copy link
Member

cclauss commented Sep 30, 2024

Closing tests_are_failing PRs to prepare for Hacktoberfest 2024

@cclauss cclauss closed this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests are failing Do not merge until tests pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants