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

Automate release with GitHub Action job #78

Merged
merged 6 commits into from
Dec 7, 2020
Merged

Conversation

aggarw13
Copy link
Contributor

@aggarw13 aggarw13 commented Dec 5, 2020

Add a GitHub Action job to trigger a release process which includes operations of:

  1. Creating and pushing the release tag to the repository
  2. Verifying the pushed tag (performing a diff with the commit ID which is tagged)
  3. Creating a ZIP for the release asset.
  4. Verifying the ZIP by performing a diff check and running unit tests on the unzipped files. 5. Creating a release on the repository for the tag, and uploading the ZIP as the release asset

The workflow can be manually triggered and takes the input values of Commit ID (to create a release for) and the Version string for tagging the release with

Testing
Here is an example run of the release job on my fork repository:
https://github.com/aggarw13/coreHTTP/runs/1512507665?check_suite_focus=true
It pushed the test tag (test-v2) and created a release along with the release asset on the repository: https://github.com/aggarw13/coreHTTP/releases/tag/test-v3

leegeth
leegeth previously approved these changes Dec 5, 2020
leegeth
leegeth previously approved these changes Dec 7, 2020
@@ -163,7 +163,11 @@ typedef struct NetworkContext NetworkContext_t;
* @param[in] pBuffer Buffer to receive the data into.
* @param[in] bytesToRecv Number of bytes requested from the network.
*
* @return The number of bytes received or a negative error code.
* @return The number of bytes received or a negative value to indicate
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change mentioned in the commit message?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, but this change was just to satisfy the SSoT CI check between transport_interface.h files of coreMQTT and coreHTTP

@gkwicker gkwicker merged commit 66dfa97 into FreeRTOS:main Dec 7, 2020
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