Skip to content

Add CICD scripts for automated deploys and QA#70

Merged
fmadjdian merged 8 commits intoaerius:mainfrom
fmadjdian:origin_add_cicd_jenkins_dev
Jul 18, 2025
Merged

Add CICD scripts for automated deploys and QA#70
fmadjdian merged 8 commits intoaerius:mainfrom
fmadjdian:origin_add_cicd_jenkins_dev

Conversation

@fmadjdian
Copy link
Contributor

Added Jenkinsfile for automated deploys and QA runs.

fmadjdian added 2 commits May 28, 2025 11:33
Also added inline comments to the files to make it more readable.
Updated the CICD files to remove comments not needed.

One thing we need to discuss; do we want the PRODUCT_NAME to be CAPS or
no caps. Only one product has CAPS in the flags target location, the
rest is non-capitalized.
@fmadjdian fmadjdian requested review from SerhatG and htetlwin945 June 4, 2025 10:48
fmadjdian added 2 commits June 5, 2025 11:01
Was accidentally included.
Updated OTA file to use external script directory for image build.
Removed QA file, will not be used for now.
@fmadjdian fmadjdian requested a review from SerhatG June 5, 2025 09:43
pom.xml for this project is not in the source folder. Adjusted the files.
Copy link
Member

@SerhatG SerhatG left a comment

Choose a reason for hiding this comment

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

LGTM mostly. Last comment will make it crash for sure, the rest is not required, just cleaner.

fmadjdian and others added 3 commits July 17, 2025 11:51
Co-authored-by: Serhat <SerhatG@users.noreply.github.com>
Co-authored-by: Serhat <SerhatG@users.noreply.github.com>
Co-authored-by: Serhat <SerhatG@users.noreply.github.com>
@fmadjdian fmadjdian requested a review from SerhatG July 17, 2025 14:32
@fmadjdian fmadjdian merged commit 3e35b2f into aerius:main Jul 18, 2025
1 check passed
@fmadjdian fmadjdian deleted the origin_add_cicd_jenkins_dev branch July 18, 2025 11:50
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.

2 participants