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

Update doxygen documentation #90

Merged
merged 16 commits into from
Nov 14, 2023
Merged

Update doxygen documentation #90

merged 16 commits into from
Nov 14, 2023

Conversation

bradleysmith23
Copy link
Contributor

@bradleysmith23 bradleysmith23 commented Nov 10, 2023

Issue #, if available:

Description of changes:
Update doxygen documentation to reflect the addition of API's to jobs.c and the addition of the Ota Job Parser

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@bradleysmith23
Copy link
Contributor Author

/bot run formatting

@bradleysmith23
Copy link
Contributor Author

/bot run formatting

@bradleysmith23
Copy link
Contributor Author

/bot run formatting

manvensh
manvensh previously approved these changes Nov 13, 2023
size_t Jobs_StartNextMsg( const char * clientToken,
size_t clientTokenLength,
char * buffer,
size_t bufferSize );
/* @[declare_jobs_startnextmsg] */
Copy link
Member

@Skptak Skptak Nov 14, 2023

Choose a reason for hiding this comment

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

Is this a duplicate line or intended?

Looks like this is already being done in other functions. Figuring out if this is needed is outside the scope of this work.

@bradleysmith23 bradleysmith23 merged commit 7f8d1d8 into aws:main Nov 14, 2023
10 checks passed
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.

4 participants