Skip to content

start writing Documentation #20

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

Merged
merged 1 commit into from
Sep 2, 2022
Merged

start writing Documentation #20

merged 1 commit into from
Sep 2, 2022

Conversation

DhruvaG2000
Copy link
Collaborator

Signed-off-by: Dhruva Gole [email protected]

@DhruvaG2000 DhruvaG2000 marked this pull request as draft August 2, 2022 06:25
@DhruvaG2000 DhruvaG2000 force-pushed the add/docs branch 2 times, most recently from 8d3d58c to d80f6d8 Compare August 31, 2022 09:37
@DhruvaG2000 DhruvaG2000 marked this pull request as ready for review August 31, 2022 09:48
@beriberikix
Copy link
Collaborator

I think we should add a link to the documentation folder from the README.

@DhruvaG2000
Copy link
Collaborator Author

I think we should add a link to the documentation folder from the README.

Done.

Comment on lines 3 to 5
**Pre-reqs:** <br>
- It is most recommended that if you have a highly custom board that even zephyr-RTOS doesn't support yet then you should add support for it first there.
- You need to satisfy all requirements mentioned in the project's README.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is confusing. Maybe remove Pre-reqs and just write a summary paragraph that indicates:

  1. Boards already supported by Zephyr can be added to the variants folder as outlined in this documentation
  2. Custom boards can first by added by following the official Zephyr porting guide. Once completed, continue here by adding a variant for your custom board.

@@ -0,0 +1,95 @@
# How to add board variants
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you think this file can be combined with variants.md?

Copy link
Collaborator

@szczys szczys left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the changes!

- specify reqd changes CMakelist

- User needs to know that they will have to change the CMakelists of their sample to find their board specific overlay

- describe how to make overlays

- Add guidance on how to map the Arduio Header in overlay files
Also rename Docs to docs to match lower case convention of other folders

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Dhruva Gole <[email protected]>

- create arduino_libs to explain using external libs
This page explains how to use an external arduino library with your project

Signed-off-by: Dhruva Gole <[email protected]>

update README: add Link to docs folder

Also add links to both golioth and my blog

Signed-off-by: Dhruva Gole <[email protected]>

- Update README: add links to the doc mds
Helps users to locate documentation more easily

Co-authored-by: Mike Szczys <[email protected]>

- Be explicit on how to add the source file to the build

Signed-off-by: Dhruva Gole <[email protected]>
Copy link
Collaborator

@beriberikix beriberikix left a comment

Choose a reason for hiding this comment

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

Looks great!

@DhruvaG2000 DhruvaG2000 merged commit c8988d8 into dev Sep 2, 2022
@DhruvaG2000 DhruvaG2000 deleted the add/docs branch September 2, 2022 17:37
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.

3 participants