search

eclipse / kapua

Eclipse Public License 2.0
222 stars 160 forks source link

Removing download of NPM and Swagger CLI for each build #3970

Closed Coduz closed 6 months ago

Coduz commented 7 months ago

This PR removes the download of NPM, Node and Swagger CLI for each build, which sometimes fails due to NPM server truncating the download.

Now you'll have to install Swagger CLI and Node separately.

brew install npm
brew install node
npm install -g @apidevtools/swagger-cli

Related Issue None

Description of the solution adopted Removed installation of NPM for each build and set OpenAPI build to use Swagger CLI and Node form the local machine (given they are configured and available in PATH)

Screenshots None

Any side note on the changes made None

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 21.19%. Comparing base (af7e915) to head (fdd6819).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse/kapua/pull/3970/graphs/tree.svg?width=650&height=150&src=pr&token=1P4N4CApH8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)](https://app.codecov.io/gh/eclipse/kapua/pull/3970?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) ```diff @@ Coverage Diff @@ ## develop #3970 +/- ## ========================================== Coverage 21.18% 21.19% Complexity 10 10 ========================================== Files 1942 1942 Lines 41266 41266 Branches 3796 3796 ========================================== + Hits 8743 8745 +2 + Misses 32118 32117 -1 + Partials 405 404 -1 ``` [see 1 file with indirect coverage changes](https://app.codecov.io/gh/eclipse/kapua/pull/3970/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)