Skip to content
This repository was archived by the owner on Sep 30, 2021. It is now read-only.

Updating README and bringing it Up-2-date #7

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

Updating README and bringing it Up-2-date #7

wants to merge 6 commits into from

Conversation

AsherThomasBabu
Copy link

Replaced old content with the latest from arduino.cc and removed broken links

Replaced old content with the latest from arduino.cc and removed broken links
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

I realize you are just copying the text from the library reference page, but I think it's worth avoiding migrating the typos along with the content.

AsherThomasBabu and others added 5 commits March 28, 2020 13:09
Co-Authored-By: per1234 <[email protected]>
Co-Authored-By: per1234 <[email protected]>
Co-Authored-By: per1234 <[email protected]>
Co-Authored-By: per1234 <[email protected]>
Co-Authored-By: per1234 <[email protected]>
@AsherThomasBabu
Copy link
Author

AsherThomasBabu commented Mar 28, 2020

I realize you are just copying the text from the library reference page, but I think it's worth avoiding migrating the typos along with the content.

@per1234 I just found that the old README was totally broken and didn't match the reference. I am sorry for the typos, I thought the content on the website might have been checked and corrected. Thanks for your time. I have committed the requested changes.

Kindly review and also check out my Proposal on the GSoC tracker. Please do give your opinion, It will help a lot in refining my Proposal further.

@per1234
Copy link
Contributor

per1234 commented Mar 28, 2020

I am sorry for the typos

No need for apologies. They're not your typos! With such a large amount of documentation to maintain, it's inevitable to find some things like this here and there.

@AsherThomasBabu
Copy link
Author

Thank you @per1234

@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

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.

3 participants