Skip to content
This repository has been archived by the owner on Mar 26, 2022. It is now read-only.

Prime numbers in reverse #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Prime numbers in reverse #7

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 23, 2016

No description provided.

@juhovan
Copy link
Owner

juhovan commented Feb 23, 2016

Feedback:

  • Separate branch and pull request for the changes, good.
    • Although .gitignore shouldn't be in this pull request
  • Try not to commit changes like removing tab on line that has nothing else, it only makes reviewing harder
  • It seems to work, although the bot will incorrectly answer 3 2 when given prime 2 but you don't have to fix it as this course is more about the tools than the code

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

Successfully merging this pull request may close these issues.

1 participant