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

Makefile: Add unit-tests target to makefile and github actions #8

Merged
merged 3 commits into from
Sep 27, 2023

Conversation

kfirtoledo
Copy link
Collaborator

@kfirtoledo kfirtoledo commented Sep 21, 2023

Add target unit-tests to the Makefile to run all unit tests and to the the PR check

@kfirtoledo kfirtoledo requested a review from orozery September 21, 2023 08:13
@kfirtoledo
Copy link
Collaborator Author

@elevran @praveingk @zivnevo

Makefile Show resolved Hide resolved
@kfirtoledo kfirtoledo force-pushed the tests branch 2 times, most recently from 7d02d44 to 61e6cc7 Compare September 26, 2023 13:20
@kfirtoledo
Copy link
Collaborator Author

@orozery ,PTAL

@kfirtoledo kfirtoledo changed the title Makefile: Add unit-tests target to makefile Makefile: Add unit-tests target to makefile and github actions Sep 26, 2023
Makefile Outdated Show resolved Hide resolved
.github/workflows/pr-check.yml Outdated Show resolved Hide resolved
Create a new prereqs-test target that contains all requirements for testing.

Signed-off-by: Kfir Toledo <[email protected]>
Add target unit-tests to the Makefile to run all unit tests

Signed-off-by: Kfir Toledo <[email protected]>
@kfirtoledo kfirtoledo force-pushed the tests branch 2 times, most recently from 88f151f to ccf3eeb Compare September 27, 2023 07:15
@kfirtoledo
Copy link
Collaborator Author

@orozery, PTAL

.github/workflows/pr-check.yml Outdated Show resolved Hide resolved
Add unit-tests job for PR check workflow in github actions

Signed-off-by: Kfir Toledo <[email protected]>
@kfirtoledo kfirtoledo merged commit 0194c74 into clusterlink-net:main Sep 27, 2023
2 checks passed
@kfirtoledo kfirtoledo deleted the tests branch September 27, 2023 08:41
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.

4 participants