Skip to content

Enabling Unit Tests in the CI/CD Pipeline - #83

Open
sdinot wants to merge 24 commits into
mainfrom
tu-enabling-in-ci
Open

sdinot wants to merge 24 commits into
mainfrom
tu-enabling-in-ci

Conversation

@sdinot

@sdinot sdinot commented Aug 13, 2026

Copy link
Copy Markdown
Member

Removed the option to skip tests during deployment.

Removed the option to skip tests during deployment.
@sdinot

sdinot commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

It is the execution of unit tests in the CI/CD pipeline that reveals the failure of certain tests, not the command-line change itself, which is not incorrect. This merge request is therefore valid and acceptable, even though its pipeline has failed.

If your project’s workflow prohibits merging under these conditions, please fix the failing tests, and I will rebase my branch.

@sdinot sdinot left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's OK for me

@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
31.8% Duplication on New Code (required ≤ 3%)
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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