Skip to content

Conversation

@furtib
Copy link
Contributor

@furtib furtib commented Sep 8, 2025

Why:
We want to test our rules on open-source projects in the CI. We want it fast and simple.

What:
Runs the local test runner in CI with pytest.

Addresses:
#20

Depends on: #73

@furtib furtib requested a review from Szelethus September 8, 2025 08:30
@furtib furtib self-assigned this Sep 8, 2025
@furtib furtib added enhancement New feature or request CI 📦 labels Sep 8, 2025
@furtib
Copy link
Contributor Author

furtib commented Sep 8, 2025

The only changed file is: foss.yaml

@furtib furtib force-pushed the ci_foss_single_threaded branch 4 times, most recently from ab57c40 to f612010 Compare September 9, 2025 13:20
Copy link
Contributor

@Szelethus Szelethus left a comment

Choose a reason for hiding this comment

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

Don't wanna be that guy, but maybe consider adding pytest separately?

@furtib furtib force-pushed the ci_foss_single_threaded branch from f612010 to b9be696 Compare September 10, 2025 06:48
@furtib furtib requested a review from Szelethus September 10, 2025 06:49
Copy link
Contributor

@Szelethus Szelethus left a comment

Choose a reason for hiding this comment

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

LGTM!

@furtib furtib force-pushed the ci_foss_single_threaded branch from b9be696 to 9faaf7a Compare September 15, 2025 08:37
@furtib furtib merged commit 7e8764a into Ericsson:main Sep 15, 2025
4 checks passed
furtib added a commit that referenced this pull request Sep 15, 2025
Why:
To demonstrate the ease a new project can be added to the CI

What:
Added a new FOSS project (zlib) to the test suite

Depends on:
#76 

Addresses:
#20
@furtib furtib changed the title Ci foss single threaded Ci FOSS - Run foss tests through the script for local testing Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI 📦 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants