-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update license check, version and pre-commit #59
Conversation
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
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
|
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
Previously both gave warnings.
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
Example License check
Old run gave warning. In new run dash output is presented.
Old:
New:
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:
Updated code tested with tags of type
v0.0.4
,0.0.5
andtarzan
. See results at https://github.com/erikbosch/vehicle-example-app2/pkgs/container/vehicle-example-app2%2Fsampleapp