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

Sys: documents corrections #285

Merged
merged 1 commit into from
Jul 1, 2022
Merged

Sys: documents corrections #285

merged 1 commit into from
Jul 1, 2022

Conversation

RocioRojas
Copy link
Contributor

@RocioRojas RocioRojas commented Jul 1, 2022

Contribution description

Make documentation corrections in the group Sys and the following modules:

-at_client
-Serialization
-Storage
-uniqueid

Testing procedure

In to the console write:

make docs

and go to doc/doxygen/html/index.html

Issues/PRs references

fixes : #281, #282

Copy link
Collaborator

@CW-75 CW-75 left a comment

Choose a reason for hiding this comment

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

This looks good for me, could you change the Capital letters in the titles of modules?

firmware/sys/doc.txt Outdated Show resolved Hide resolved
firmware/sys/doc.txt Outdated Show resolved Hide resolved
Copy link
Collaborator

@CW-75 CW-75 left a comment

Choose a reason for hiding this comment

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

Looks good for me, merge it

@RocioRojas RocioRojas merged commit 3789ee2 into Mesh4all:main Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sys/Serialization: Serialization needs an update in the doc.txt
2 participants