Skip to content

Conversation

thodson-usgs
Copy link

Clean up language in README.md

@ellieLitwack
Copy link

I'll review this!

@@ -1,13 +1,15 @@
Ok so I guess you are reading this cuz you wanna use my code. There are some
functions that do stuf and thats:
The PR_tutorial package contains some simple functions to demonstrate the process of creating and merging a PR request.
Copy link

@ellieLitwack ellieLitwack Jul 14, 2023

Choose a reason for hiding this comment

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

This is much more professional - thank you for making that change!

and this other part does something. I forget why that I did it:
Another function computes the Fibonacci sequence:

>>> fibonnaccci(100)
Copy link

@ellieLitwack ellieLitwack Jul 14, 2023

Choose a reason for hiding this comment

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

I think it would be helpful to know how to import this function.

Copy link
Author

Choose a reason for hiding this comment

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

Good catch.

@thodson-usgs
Copy link
Author

I believe I've addressed all your comments. Thanks for the great tutorial!

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