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

Run unit tests for Linux on CI #538

Merged
merged 3 commits into from
Sep 26, 2024
Merged

Conversation

vifactor
Copy link
Collaborator

@vifactor vifactor commented Sep 16, 2024

This adds a step into Linux pipeline to build and run unit tests

@vifactor vifactor marked this pull request as ready for review September 16, 2024 19:59
@vifactor
Copy link
Collaborator Author

@hannesa2 this MR on build jobs that I certainly did not touch. Is this a known issue?

@hannesa2
Copy link
Collaborator

Last master builds properly https://github.com/COVESA/dlt-viewer/actions

But I see, when running from a fork the currently necessary variables for notarization are missing
@bunty95

@hannesa2
Copy link
Collaborator

After merge #539 it should work for you as well.
Please simply rebase

@hannesa2 hannesa2 added the Build pipeline Build pipeline for CI label Sep 17, 2024
@hannesa2 hannesa2 changed the title Build unit tests on CI Run unit tests for Linux on CI Sep 17, 2024
@alexmucde
Copy link
Collaborator

@vifactor Is this ready for merge? Perhaps you can set the merge label, when the PR is ready for merge.

@alexmucde alexmucde added this to the Release v2.27.0 milestone Sep 23, 2024
@vifactor
Copy link
Collaborator Author

@alexmucde yes, this is ready to merge. regarding labels, I do not think I have rights to add them

@alexmucde alexmucde merged commit 1cfacb5 into COVESA:master Sep 26, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build pipeline Build pipeline for CI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants