Skip to content

replace integration tests make targets with bazel targets - #610

Merged
SurajBDeore merged 1 commit into
mainfrom
feature/integration-tests-bazel
Aug 5, 2026
Merged

replace integration tests make targets with bazel targets#610
SurajBDeore merged 1 commit into
mainfrom
feature/integration-tests-bazel

Conversation

@SurajBDeore

@SurajBDeore SurajBDeore commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Converted Make targets to Bazel: Replaced Make-based integration tests with four Bazel targets for coverage traceability tests (coverage, coverage_half, coverage_mix, coverage_zero), improving build system integration and test reproducibility.

@SurajBDeore
SurajBDeore requested a review from a team as a code owner July 13, 2026 09:49
@SurajBDeore
SurajBDeore force-pushed the feature/integration-tests-bazel branch from 6f1840b to 76f854e Compare July 13, 2026 09:50
@kedarnn kedarnn added the internal Affects the CI, tests or refactorings only, not relevant to the end-user label Jul 15, 2026
@mugdhadhole1

mugdhadhole1 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Please add description for this PR some detail about what is happening in this PR

@mugdhadhole1

Copy link
Copy Markdown
Contributor

Replacing the target in CI is also necessary and it should work in CI
make integration-tests

@mugdhadhole1 mugdhadhole1 changed the title add Bazel targets for integration-test replace integration tests make targets with bazel targets Jul 23, 2026
@SurajBDeore
SurajBDeore force-pushed the feature/integration-tests-bazel branch 9 times, most recently from 31401c1 to ca23d5e Compare July 23, 2026 13:14
Comment thread Makefile
Comment thread Makefile
@SurajBDeore
SurajBDeore force-pushed the feature/integration-tests-bazel branch 2 times, most recently from d37c00c to 6974d01 Compare July 28, 2026 09:18

@mugdhadhole1 mugdhadhole1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Now that we have the bazel targets which replace the make files in the respective folders.
The make files should be deleted.

Comment thread .github/workflows/ci.yml Outdated
Comment thread Makefile
@SurajBDeore
SurajBDeore force-pushed the feature/integration-tests-bazel branch 3 times, most recently from d8abf14 to 36dec59 Compare August 5, 2026 08:17
@SurajBDeore
SurajBDeore force-pushed the feature/integration-tests-bazel branch from 36dec59 to 2d96d3e Compare August 5, 2026 08:19
@SurajBDeore
SurajBDeore merged commit 2869311 into main Aug 5, 2026
27 checks passed
@SurajBDeore
SurajBDeore deleted the feature/integration-tests-bazel branch August 5, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Affects the CI, tests or refactorings only, not relevant to the end-user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants