Skip to content

Conversation

brepowell
Copy link

No description provided.

@tntmatthews
Copy link

I will be reviewing

Ok so I guess you are reading this cuz you wanna use my code. There are some
functions that do stuf and thats:
You are reading this because you want to use my code.
Check out this function. This will import a factorial:

Choose a reason for hiding this comment

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

instead of "a factorial" you could write something like "the factorial function"

Choose a reason for hiding this comment

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

This looks okay - but perhaps you could expand further

9

and this other part does something. I forget why that I did it:
This functino will make a fibonnacci sequence up to 100:

Choose a reason for hiding this comment

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

small typo "functino" -> "function"

@nikoladze
Copy link

I will be reviewing

sorry - saw that too late ;)

Copy link

@tntmatthews tntmatthews left a comment

Choose a reason for hiding this comment

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

this wraps up my comments

Ok so I guess you are reading this cuz you wanna use my code. There are some
functions that do stuf and thats:
You are reading this because you want to use my code.
Check out this function. This will import a factorial:

Choose a reason for hiding this comment

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

This looks okay - but perhaps you could expand further

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.

3 participants