-
-
Notifications
You must be signed in to change notification settings - Fork 178
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
Minor cleanup #129
Minor cleanup #129
Conversation
The current file was missing the "How to Apply These Terms to Your New Programs" section, which is an integral part of the GPL and is not supposed to be removed: https://www.gnu.org/licenses/gpl-faq.html#GPLOmitPreamble Correct file was downloaded from: https://www.gnu.org/licenses/gpl-3.0.txt
Thanks! I'll review and merge tomorrow. |
Absolutely no rush :) Thank you for the review and for making ffsend. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- The double space at the end of lines in Markdown files are intentional to insert a line break.
- The name
reqwest
is intentional as it's the name of the backing library.
Other than that, all your changes are great!
I've marked each of the instances with a comment as per above. Would you mind to revert these cases? I'd be happy to merge after.
Edit: this spawned a lot of comments, oops.
Thanks for your work! Two things were intentional however, please see the review above. |
Thanks for catching that in the review, I addressed all of it and rebased. |
Thanks again! Merged. |
No description provided.