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 license check, version and pre-commit #59

Merged
merged 2 commits into from
Jun 26, 2024

Conversation

erikbosch
Copy link
Contributor

@erikbosch erikbosch commented Jun 14, 2024

Previously both gave warnings.

  • Pre-commit as using Node.js 16
  • License check as error code was presented

Fixes #58
Fixes #57

Changes tested (for python app) on this repo https://github.com/erikbosch/vehicle-example-app2

Example pre-commit

Warning only present in old build

Run unit tests and linters
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Example License check

Old run gave warning. In new run dash output is presented.

Old:

Check Software Licenses
Process completed with exit code 23.

New:

[main] INFO Querying Eclipse Foundation for license data for 79 items.
[main] INFO Found 23 items.
[main] INFO Querying ClearlyDefined for license data for 57 items.
[main] INFO Found 57 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO git/github/actions/download-artifact/v4
[main] INFO git/github/actions/setup-java/v4
[main] INFO git/github/actions/setup-node/v4
[main] INFO git/github/actions/setup-python/v5
[main] INFO git/github/actions/upload-artifact/v4
[main] INFO git/github/aquasecurity/trivy-action/0.19.0
[main] INFO git/github/dawidd6/action-download-artifact/v3
[main] INFO git/github/docker/build-push-action/v5
[main] INFO git/github/docker/setup-buildx-action/v3
[main] INFO git/github/docker/setup-qemu-action/v3
[main] INFO git/github/fountainhead/action-wait-for-check/v1.2.0
[main] INFO git/github/mikepenz/action-junit-report/v4
[main] INFO git/github/softprops/action-gh-release/v2
[main] INFO pypi/pypi/-/coverage/7.3.2
[main] INFO pypi/pypi/-/exceptiongroup/1.1.3
[main] INFO pypi/pypi/-/grpcio/1.59.0
[main] INFO pypi/pypi/-/mypy/1.5.1
[main] INFO pypi/pypi/-/paho-mqtt/1.6.1
[main] INFO pypi/pypi/-/parameterized/0.9.0
[main] INFO pypi/pypi/-/setuptools/58.1.0
[main] INFO pypi/pypi/-/typing-extensions/4.11.0
[main] INFO pypi/pypi/-/velocitas-sdk/0.14.1
[main] INFO pypi/pypi/-/wheel/0.41.2
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.

Changes inspired by what exist in https://github.com/eclipse-kuksa/kuksa-actions/tree/main/check-dash. If a token is available then one could consider supporting automatic review requests as well.

Example Battila7

Previously we got warnings like:

image

Updated code tested with tags of type v0.0.4, 0.0.5 and tarzan. See results at https://github.com/erikbosch/vehicle-example-app2/pkgs/container/vehicle-example-app2%2Fsampleapp

@erikbosch erikbosch marked this pull request as ready for review June 14, 2024 09:15
@erikbosch
Copy link
Contributor Author

Here we go again, two more ideas to get rid of warnings. Tested only on Python app, but have difficulties seeing that this should give problems for cpp/sdks

Previously both gave warnings.
Pre-commit as using Node.js 16
License check as error code was presented

Remove abandoned action

Action is no longer maintained, give Node.js warning

Fixes eclipse-velocitas#57
@erikbosch erikbosch changed the title Update license check and pre-commit Update license check, version and pre-commit Jun 14, 2024
@erikbosch erikbosch requested review from MP91 and BjoernAtBosch June 25, 2024 11:57
src/common/workflows/check-licenses.yml Outdated Show resolved Hide resolved
src/app/workflows/gen-desired-state.yml Outdated Show resolved Hide resolved
@MP91
Copy link
Contributor

MP91 commented Jun 25, 2024

We also have the VELOCITAS_PROJECT_TOKEN which should do the stuff for dash

@erikbosch
Copy link
Contributor Author

We also have the VELOCITAS_PROJECT_TOKEN which should do the stuff for dash

You mean to request a review, similar to how we use secrets.ECLIPSE_GITLAB_API_TOKEN in for example Databroker. I think that makes sense, but should we do it in this PR?


      - name: Dash license check
        uses: eclipse-kuksa/kuksa-actions/check-dash@4
        with:
          dashinput: ${{github.workspace}}/dash-databroker-deps
          dashtoken: ${{ secrets.ECLIPSE_GITLAB_API_TOKEN }}

Copy link
Contributor

@MP91 MP91 left a comment

Choose a reason for hiding this comment

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

lgtm

@erikbosch erikbosch merged commit ba785bc into eclipse-velocitas:main Jun 26, 2024
3 checks passed
@erikbosch erikbosch deleted the erik_warn branch July 1, 2024 06:45
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.

[Bug]: CI warnings [Feature]: Replace get-version-action
2 participants