Skip to content

feat/generic way to run tests in both VSCode and Github actions#116

Open
farah-t-trigui wants to merge 3 commits intomainfrom
feat/variablize-and-add-test-yml-ressources
Open

feat/generic way to run tests in both VSCode and Github actions#116
farah-t-trigui wants to merge 3 commits intomainfrom
feat/variablize-and-add-test-yml-ressources

Conversation

@farah-t-trigui
Copy link
Contributor

A Github Action workflow that ensures code quality and security before any merge by running unit tests with JaCoCo coverage reporting, static code analysis via SonarQube, CVE and secret scanning via Trivy, and git history secret detection via Gitleaks. Results are published as a step summary in the GitHub Actions UI, artifacts are retained for 14 days, and live badges are pushed to a public Gist for README display.
This workflow will be triggered on push to every branch and on PR to main branch.

@farah-t-trigui farah-t-trigui requested a review from eskenazit March 9, 2026 09:30
@farah-t-trigui farah-t-trigui force-pushed the feat/variablize-and-add-test-yml-ressources branch from 03e2347 to 139160d Compare March 9, 2026 09:40
@farah-t-trigui farah-t-trigui force-pushed the feat/variablize-and-add-test-yml-ressources branch 3 times, most recently from c715b86 to 47036d0 Compare March 9, 2026 15:54
@eskenazit
Copy link
Contributor

@jlouvel the src/test/resources/avro-capability.yaml file seems out of date and is blocking merge. Do you want to update it to 0.5 specification or remove it ?

@farah-t-trigui farah-t-trigui force-pushed the feat/variablize-and-add-test-yml-ressources branch from 47036d0 to 749ef8c Compare March 10, 2026 07:49
@farah-t-trigui farah-t-trigui force-pushed the feat/variablize-and-add-test-yml-ressources branch from 749ef8c to 7c7f337 Compare March 10, 2026 07:50
Copy link
Contributor

@eskenazit eskenazit left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants