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.

Sign up for GitHub

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

Jump to bottom

[TEST]: RUN CI #309

Closed
wants to merge 41 commits into from
Closed

[TEST]: RUN CI #309

wants to merge 41 commits into from

Conversation

AniruddhaKanhere
Copy link
Member

Description

Test Steps

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

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

Dakshit Babbar and others added 30 commits September 6, 2024 10:53
Update the APIs to check the connected flag and be thread safe
f79d06b
Add an intermediate connection state to handle disconnects due to net…
1e2cd7a 
…work failure
Rectify Unit Tests Errors
19fd967
Add a new macro to detect transport error in core_mqtt_serializer.h
5ead3f8
Resolve Spellcheck and Doxygen Tests
431b34f
Resolve Formatting Test
f67688e
@DakshitBabbar
Fix Unit Tests
15c6cbb
@actions-user
Uncrustify: triggered by comment.
0eda275
@DakshitBabbar
Add new unit tests for 100% code coverage
eb68f0c
@DakshitBabbar
Merge branch 'main' into coreMQTTThreadSafe
022fcd1
Resolve Formatting and PR Comments
0d7dade
@DakshitBabbar
Implement critical section without mutexTaken flag
5c3524e
@DakshitBabbar
Rectify conflicts
2382195
Resolve issues with failing unit tests on Mac and Memory Statistics
b485375
@DakshitBabbar
Merge branch 'FreeRTOS:main' into coreMQTTThreadSafe
febdaa0
Resolve Formatting
8ca97ab
Enable the library to re-transmit publishes on unclean session connec…
dbef1fd 
…tion
Add changes for new return status codes and make retransmit thread safe
ad42032
Update the case when re-transmits fail after connack is received
02d3456
@actions-user
Uncrustify: triggered by comment.
9e8c309
Update unit tests
8fa59a1
Updated existing unit tests
86c6b9f
Implement new unit tests for the changes related to publish retransmit
4c86ac2
@DakshitBabbar
Merge branch 'FreeRTOS:main' into reTransmit
9413942
Merge main into reTransmit
853b6f4
Clear publish copy after state update on receiving ACK
e5834d8
@actions-user
Uncrustify: triggered by comment.
0753992
Resolve memory stats, spell check, doxygen example and comments
43aed2a
Resolve formating issues
80b5931
Resolve PR comments
8e04527
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AniruddhaKanhere @DakshitBabbar @actions-user