-
Notifications
You must be signed in to change notification settings - Fork 60
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
Modify package generation actions #157
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## 4.9.0 #157 +/- ##
=======================================
Coverage 66.96% 66.96%
=======================================
Files 3300 3300
Lines 63585 63585
Branches 10141 10141
=======================================
Hits 42582 42582
Misses 18538 18538
Partials 2465 2465
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
724d1fa
to
acd6429
Compare
run: | | ||
cd ${{ env.CURRENT_DIR }}/dev-tools/build-packages | ||
bash ./build-packages.sh \ | ||
-v ${{ env.VERSION }} \ | ||
-r ${{ env.REVISION }} \ | ||
-r ${{ inputs.revision }} \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the revision be the same as the one in the package.json?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR aims to modify the workflow that generates the Wazuh Dashboard packages.
Evidence
The workflow now generates one package at a time. It can have dev or prod nomenclature, and it is possible to upload or not the sha512 of the package.
If the inputs are invalid, the worklflow will stop and no package will be generated
Issues Resolved
#156
Check List
yarn test:jest
yarn test:jest_integration