Skip to content

4 Readme Changes#21

Open
Thovthe wants to merge 1 commit intodanjulio:masterfrom
Thovthe:patch-1
Open

4 Readme Changes#21
Thovthe wants to merge 1 commit intodanjulio:masterfrom
Thovthe:patch-1

Conversation

@Thovthe
Copy link
Copy Markdown

@Thovthe Thovthe commented Mar 18, 2022

Added:
A table of contents
A community software section that should allow closing Issue #8
A statement of general licensing intent

The diff looks a little weird at the bottom but I just formatted the name of RaspberryPi

Added:
A table of contents
A community software section that should allow closing Issue danjulio#8 
A statement of general licensing intent
@Thovthe Thovthe changed the title 3 Readme Changes 4 Readme Changes Mar 18, 2022
@danjulio
Copy link
Copy Markdown
Owner

Thanks for the suggestions. I have the following comments, questions about this PR.

  1. I'm not sure why that page needs a TOC. I don't see that it adds anything to navigation. Perhaps just additional wording in the first paragraph indicating the repository contains code and designs for the various platforms I have experimented with (honestly I should have split this into a bunch of repositories but don't want to know because of all the links pointing here).
  2. For the licensing section I think I'd rather say that there are a variety of licenses in use, including "as-is" and perhaps a note that some licenses are TBD.
  3. We should be explicit that the R-package relates to the ESP32 directory.

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