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 release action for version number #297

Merged
merged 22 commits into from
Jul 11, 2024

Conversation

chinglee-iot
Copy link
Member

@chinglee-iot chinglee-iot commented Jul 10, 2024

Description

  • Update the release action for version number include the following files
    • docs/doxygen/config.doxyfile - PROJECT_NUMBER
    • manifest.yml file - version
    • source file - version header
    • core_mqtt.h - version number
  • Add version number check in "Create ZIP and verify package for release asset" steps. Including the following
    • docs/doxygen/config.doxyfile - PROJECT_NUMBER
    • manifest.yml file - version
    • source file - version header
    • core_mqtt.h - version number
  • Update all the version number to "v2.3.0+" and "<DEVELOPMENT BRANCH>"

Test Steps

Using release action to create release should update the following

  • source/include/core_mqtt.h version number
  • source files header version number
  • doxygen version number
  • manifest.yml number
  • SBOM file

Tested in personal fork without problem : https://github.com/FreshDevGo/coreMQTT/actions/runs/9885707328/job/27304218049

Test with wrong source file version number : https://github.com/FreshDevGo/coreMQTT/actions/runs/9885727002/job/27304274003
Test with wrong manifest.yml version number : https://github.com/FreshDevGo/coreMQTT/actions/runs/9885726029/job/27304270303
Test with wrong doxygen version number : https://github.com/FreshDevGo/coreMQTT/actions/runs/9885723302/job/27304269170
Test with wrong version number macro in core_mqtt.h : https://github.com/FreshDevGo/coreMQTT/actions/runs/9885724835/job/27304268841

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.

amazonKamath
amazonKamath previously approved these changes Jul 10, 2024
@amazonKamath amazonKamath merged commit 9b993a6 into FreeRTOS:main Jul 11, 2024
11 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