Skip to content
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

README: convert to markdown #17

Merged
merged 1 commit into from
Jun 12, 2019
Merged

README: convert to markdown #17

merged 1 commit into from
Jun 12, 2019

Conversation

z3ntu
Copy link
Collaborator

@z3ntu z3ntu commented Jun 9, 2019

Ref #11

This is a plain conversion without any changes to the text.

Preview: https://github.com/z3ntu/hidapi/blob/readme_md/README.md

@todbot
Copy link
Contributor

todbot commented Jun 9, 2019

Looks good to me. Thanks!

Copy link
Member

@Qbicz Qbicz left a comment

Choose a reason for hiding this comment

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

Thank you. Looking at the preview, I have 2 observations:

  • the text is now rather lengthy in comparison to original, which is more compressed. I feel the markdown version is overwhelming, what about having some table of contents in the beginning?
  • Title of library is broken down into 2 lines, with "X" from OS X in second line. What about using ## heading which would results in smaller letters?

Copy link
Member

@Youw Youw left a comment

Choose a reason for hiding this comment

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

+1 to table of contents

Title of library is broken down into 2 lines, with "X" from OS X in second line.

Not on my monitor, but good point. See my suggestion.

README.md Show resolved Hide resolved
@Youw Youw merged commit fdae95d into libusb:master Jun 12, 2019
Qbicz pushed a commit that referenced this pull request Jun 14, 2019
The 'make' build on Linux/MacOS generates an error since README.txt was renamed to README.md. See #17 .
@z3ntu z3ntu deleted the readme_md branch June 20, 2019 17:14
@mcuee mcuee added the documentation Improvements or additions to documentation label Jul 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants