fix(testing): Breaking changes should not break ci by evrardjp · Pull Request #5739 · external-secrets/external-secrets

evrardjp · 2025-12-17T10:25:34Z

Without this, a ! in commit title will break labeler.

This is a problem, as it means manually labeling issues.

This fixes it by correcting the github action.

Problem Statement

What is the problem you're trying to solve?

Related Issue

Fixes #...

Proposed Changes

How do you like to solve the issue and why?

Format

Please ensure that your PR follows the following format for the title:

feat(scope): add new feature
fix(scope): fix bug
docs(scope): update documentation
chore(scope): update build tool or dependencies
ref(scope): refactor code
clean(scope): provider cleanup
test(scope): add tests
perf(scope): improve performance
desig(scope): improve design

Where scope is optionally one of:

charts
release
testing
security
templating

Checklist

I have read the contribution guidelines
All commits are signed with git commit --signoff
My changes have reasonable test coverage
All tests pass with make test
I ensured my PR is ready for review with make reviewable

Summary by CodeRabbit

Chores
- PR workflow now automatically applies a "breaking-change" label for identified breaking changes, improving reviewer visibility and release planning.
- Logging around breaking-change detection remains to aid troubleshooting of PR labeling behavior.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2025-12-17T10:25:45Z

Walkthrough

Replaces use of the BREAKING_CHANGE_LABEL environment variable with the literal string "breaking-change" when applying the breaking-change label in the PR-labeling GitHub Actions workflow; existing debug/log messages and conditional trigger for breaking-change titles remain unchanged.

Changes

Cohort / File(s)	Summary
GitHub Actions workflow `\.github/workflows/pull-request-label\.yml`	Replaced pushing the `BREAKING_CHANGE_LABEL` environment variable with pushing the literal string `"breaking-change"` when applying the breaking-change label; existing debug/log messages and conditional logic for detecting breaking-change titles left intact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Single-line/single-file change in a workflow.
Review points:
- Ensure the label string is correctly quoted and passed in the action step.
- Confirm debug logs and conditionals still reference the intended variables or messages.

Poem

🐇 I nibbled YAML, found a name,
No more env — a literal claim,
"breaking-change" now hops in view,
I logged a cheer and bounced on through,
✨ Tiny tweak — the label's true.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The PR description includes the required template sections, but Problem Statement, Related Issue, and Proposed Changes are largely unfilled, leaving key details undocumented.	Complete the Problem Statement, Related Issue, and Proposed Changes sections with specific details about the issue and solution instead of leaving them blank.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title 'fix(testing): Breaking changes should not break ci' directly addresses the main change—fixing the GitHub Actions labeler so that exclamation marks in commit titles don't break CI. This aligns with the file changes in the workflow and the PR objective.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

.github/workflows/pull-request-label.yml (1)
148-157: Add "breaking-change" to the qualifying labels list.

The comment on line 25 explicitly states that extensions should be included in the verify-labels step. However, the "breaking-change" label is not present in the any_of list on line 156. This will cause the verify-labels job to fail for PRs that only have a breaking-change label (and potentially scope labels), even though they were correctly labeled by the conventional-commit-labeler job.

Apply this diff to include the breaking-change label:
-          any_of: kind/chore,kind/bug,kind/feature,kind/dependency,kind/refactor,kind/design,kind/documentation,kind/testing,kind/performance,kind/cleanup
+          any_of: kind/chore,kind/bug,kind/feature,kind/dependency,kind/refactor,kind/design,kind/documentation,kind/testing,kind/performance,kind/cleanup,breaking-change

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c3bd596 and c72c1ff.

📒 Files selected for processing (1)

.github/workflows/pull-request-label.yml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)

GitHub Check: publish-artifacts (Dockerfile, CGO_ENABLED=0, amd64 arm64 s390x ppc64le, linux/amd64,linux/arm64,... / Build and Publish
GitHub Check: publish-artifacts (Dockerfile.ubi, CGO_ENABLED=0, amd64 arm64 ppc64le, linux/amd64,linux/arm64,li... / Build and Publish
GitHub Check: publish-artifacts (Dockerfile.ubi, CGO_ENABLED=0 GOEXPERIMENT=boringcrypto, amd64 ppc64le, linux/... / Build and Publish
GitHub Check: lint
GitHub Check: unit-tests
GitHub Check: check-diff
GitHub Check: Analyze project (actions, none)
GitHub Check: Analyze project (go, autobuild)

🔇 Additional comments (1)

.github/workflows/pull-request-label.yml (1)

38-38: LGTM! This fixes the missing environment variable.

The addition correctly provides the BREAKING_CHANGE_LABEL value that was already being referenced on line 79, fixing the bug where breaking changes weren't being labeled properly.

coderabbitai

Actionable comments posted: 2

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c72c1ff and 74f91a1.

📒 Files selected for processing (1)

.github/workflows/pull-request-label.yml (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)

GitHub Check: publish-artifacts (Dockerfile.ubi, CGO_ENABLED=0 GOEXPERIMENT=boringcrypto, amd64 ppc64le, linux/... / Build and Publish
GitHub Check: publish-artifacts (Dockerfile, CGO_ENABLED=0, amd64 arm64 s390x ppc64le, linux/amd64,linux/arm64,... / Build and Publish
GitHub Check: publish-artifacts (Dockerfile.ubi, CGO_ENABLED=0, amd64 arm64 ppc64le, linux/amd64,linux/arm64,li... / Build and Publish
GitHub Check: lint
GitHub Check: check-diff
GitHub Check: unit-tests
GitHub Check: Analyze project (go, autobuild)
GitHub Check: Analyze project (actions, none)

🔇 Additional comments (1)

.github/workflows/pull-request-label.yml (1)

77-81: Clarify the bug fix described in the PR.

The PR description states this "fixes an issue where a '!' in a commit title breaks the GitHub Actions labeler," but the regex at line 64 already correctly handles the "!" character with (?<breaking>!?). This change appears to be refactoring the hardcoded label name to use an environment variable rather than fixing a parsing bug.

Can you clarify:

What specifically was breaking before this change?

Was the issue that the "breaking-change" label didn't exist in the repository?

Or was there a typo in the previously hardcoded label name?

Understanding the root cause will help ensure this change fully addresses the issue.

coderabbitai · 2025-12-17T10:35:31Z

.github/workflows/pull-request-label.yml

              "test":"kind/testing",
              "perf":"kind/performance"
            }
+          BREAKING_CHANGE_LABEL: "breaking-change"


⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

# First, locate the file and read it fd "pull-request-label.yml" -t f # Show its contents with line numbers cat -n .github/workflows/pull-request-label.yml

Repository: external-secrets/external-secrets

Length of output: 7401

Remove debug console.log statement at line 79.

Externalizing the breaking-change label name as an environment variable is good practice and maintains consistency with the TYPE_TO_LABEL pattern above. However, line 79 contains a temporary debug statement with a // debug jp comment that must be removed before merging.

🤖 Prompt for AI Agents

.github/workflows/pull-request-label.yml around line 38 and specifically remove the temporary debug statement at line 79; there is a console.log with a "// debug jp" comment that should be deleted. Edit the workflow to remove that console.log line and the trailing comment, leaving only the intended environment variable and workflow logic intact, and ensure no other leftover debug prints remain before committing.

coderabbitai · 2025-12-17T10:35:31Z

.github/workflows/pull-request-label.yml

              if (breaking) {
                console.log("Adding breaking change label");
+                console.log(process.env.BREAKING_CHANGE_LABEL); // debug jp
                labels.push(process.env.BREAKING_CHANGE_LABEL);
              }


🛠️ Refactor suggestion | 🟠 Major

Remove temporary debug statement.

The debug log on line 79 with the comment // debug jp appears to be temporary debugging code that should be removed before merging.

Apply this diff to remove the debug statement:

if (breaking) { console.log("Adding breaking change label"); - console.log(process.env.BREAKING_CHANGE_LABEL); // debug jp labels.push(process.env.BREAKING_CHANGE_LABEL); }

Alternatively, if logging the label value is needed for troubleshooting, make it more descriptive:

if (breaking) { - console.log("Adding breaking change label"); - console.log(process.env.BREAKING_CHANGE_LABEL); // debug jp + console.log(`Adding breaking change label: ${process.env.BREAKING_CHANGE_LABEL}`); labels.push(process.env.BREAKING_CHANGE_LABEL); }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

if (breaking) {

console.log("Adding breaking change label");

console.log(process.env.BREAKING_CHANGE_LABEL); // debug jp

labels.push(process.env.BREAKING_CHANGE_LABEL);

}

if (breaking) {

console.log("Adding breaking change label");

labels.push(process.env.BREAKING_CHANGE_LABEL);

}

🤖 Prompt for AI Agents

.github/workflows/pull-request-label.yml lines 77-81: remove the temporary debug console.log on line 79 that prints process.env.BREAKING_CHANGE_LABEL (the one with "// debug jp"); simply delete that debug statement so only the purposeful log and label push remain, or if you need to retain logging for troubleshooting replace it with a single descriptive log like `console.log("Breaking change label:", process.env.BREAKING_CHANGE_LABEL)` and remove the inline comment.

Without this, a ! in commit title will break labeler. This is a problem, as it means manually labeling issues. This fixes it by correcting the github action. Signed-off-by: Jean-Philippe Evrard <open-source@a.spamming.party>

sonarqubecloud · 2025-12-17T10:42:57Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

This PR contains the following updates: | Package | Update | Change | |---|---|---| | [external-secrets](https://github.com/external-secrets/external-secrets) | minor | `1.1.1` -> `1.2.0` | --- ### Release Notes <details> <summary>external-secrets/external-secrets (external-secrets)</summary> ### [`v1.2.0`](https://github.com/external-secrets/external-secrets/releases/tag/v1.2.0) [Compare Source](external-secrets/external-secrets@v1.1.1...v1.2.0) Image: `ghcr.io/external-secrets/external-secrets:v1.2.0` Image: `ghcr.io/external-secrets/external-secrets:v1.2.0-ubi` Image: `ghcr.io/external-secrets/external-secrets:v1.2.0-ubi-boringssl`  #### What's Changed ##### General - chore: bump 1.1.1 by [@gusfcarvalho](https://github.com/gusfcarvalho) in [#5687](external-secrets/external-secrets#5687) - chore: fix the argocd e2e test case by [@Skarlso](https://github.com/Skarlso) in [#5688](external-secrets/external-secrets#5688) - feat(provider): add Barbican provider support by [@rkferreira](https://github.com/rkferreira) in [#5398](external-secrets/external-secrets#5398) - docs(secretserver): promote secretserver provider to beta by [@DelineaSahilWankhede](https://github.com/DelineaSahilWankhede) in [#5668](external-secrets/external-secrets#5668) - feat(controller): add flag to enable/disable secretstore reconcile by [@Ilhan-Personal](https://github.com/Ilhan-Personal) in [#5653](external-secrets/external-secrets#5653) - fix(aws-secrets-manager): Apply filtering based on both name and tags if provided by [@iypetrov](https://github.com/iypetrov) in [#5685](external-secrets/external-secrets#5685) - fix(gcpsm): SecretExists should check for regional secrets when store location is specified by [@tokiwong](https://github.com/tokiwong) in [#5708](external-secrets/external-secrets#5708) - feat: introduce store deprecation by [@gusfcarvalho](https://github.com/gusfcarvalho) in [#5711](external-secrets/external-secrets#5711) - feat(charts): add global values for common deployment configurations by [@Gabryel8818](https://github.com/Gabryel8818) in [#5652](external-secrets/external-secrets#5652) - feat: add Doppler OIDC-based authentication by [@mikesellitto](https://github.com/mikesellitto) in [#5475](external-secrets/external-secrets#5475) - fix: make custom configuration available regardless of environment by [@Skarlso](https://github.com/Skarlso) in [#5713](external-secrets/external-secrets#5713) - chore(chart): update bitwarden dependency to v0.5.2 by [@Skarlso](https://github.com/Skarlso) in [#5719](external-secrets/external-secrets#5719) - docs(templating): update rbac for generic targets by [@lostick](https://github.com/lostick) in [#5736](external-secrets/external-secrets#5736) - fix(testing): Breaking changes should not break ci by [@evrardjp](https://github.com/evrardjp) in [#5739](external-secrets/external-secrets#5739) - fix(security): Get rid of getSecretKey by [@evrardjp](https://github.com/evrardjp) in [#5738](external-secrets/external-secrets#5738) - fix(aws): parse resource policies into canonical JSON (sorted) before comparing by [@cmoscofian](https://github.com/cmoscofian) in [#5622](external-secrets/external-secrets#5622) - docs: Fix example in GCP documentation by [@headcr4sh](https://github.com/headcr4sh) in [#5745](external-secrets/external-secrets#5745) - chore(secretserver): update dependencies to accept new DelineaXPM/tss-sdk-go by [@DelineaSahilWankhede](https://github.com/DelineaSahilWankhede) in [#5742](external-secrets/external-secrets#5742) - fix(gcpsm): Improve SecretExists method in GCP secret manager provider by [@tosih](https://github.com/tosih) in [#5610](external-secrets/external-secrets#5610) - chore(docs): add clarification to helm values being disabled by [@Skarlso](https://github.com/Skarlso) in [#5746](external-secrets/external-secrets#5746) - fix(release): apply [`64dc681`](external-secrets/external-secrets@64dc681) to release by [@jakobmoellerdev](https://github.com/jakobmoellerdev) in [#5749](external-secrets/external-secrets#5749) - docs(release): 1.2 stability-support.md by [@jakobmoellerdev](https://github.com/jakobmoellerdev) in [#5750](external-secrets/external-secrets#5750) ##### Dependencies - chore(deps): bump golang from 1.25.4 to 1.25.5 by [@dependabot](https://github.com/dependabot)\[bot] in [#5693](external-secrets/external-secrets#5693) - chore(deps): bump golang from 1.25.4-bookworm to 1.25.5-bookworm in /e2e by [@dependabot](https://github.com/dependabot)\[bot] in [#5702](external-secrets/external-secrets#5702) - chore(deps): bump ubi9/ubi from `dcd8128` to `75937d9` by [@dependabot](https://github.com/dependabot)\[bot] in [#5655](external-secrets/external-secrets#5655) - chore(deps): bump peter-evans/slash-command-dispatch from 5.0.0 to 5.0.1 by [@dependabot](https://github.com/dependabot)\[bot] in [#5695](external-secrets/external-secrets#5695) - chore(deps): bump github/codeql-action from 4.31.5 to 4.31.7 by [@dependabot](https://github.com/dependabot)\[bot] in [#5696](external-secrets/external-secrets#5696) - chore(deps): bump actions/stale from 10.1.0 to 10.1.1 by [@dependabot](https://github.com/dependabot)\[bot] in [#5697](external-secrets/external-secrets#5697) - chore(deps): bump actions/create-github-app-token from 2.2.0 to 2.2.1 by [@dependabot](https://github.com/dependabot)\[bot] in [#5700](external-secrets/external-secrets#5700) - chore(deps): bump step-security/harden-runner from 2.13.2 to 2.13.3 by [@dependabot](https://github.com/dependabot)\[bot] in [#5698](external-secrets/external-secrets#5698) - chore(deps): bump actions/checkout from 6.0.0 to 6.0.1 by [@dependabot](https://github.com/dependabot)\[bot] in [#5699](external-secrets/external-secrets#5699) - chore(deps): bump platformdirs from 4.5.0 to 4.5.1 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5705](external-secrets/external-secrets#5705) - chore(deps): bump distroless/static from `87bce11` to `4b2a093` by [@dependabot](https://github.com/dependabot)\[bot] in [#5692](external-secrets/external-secrets#5692) - chore(deps): bump alpine from 3.22 to 3.23 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5703](external-secrets/external-secrets#5703) - chore(deps): bump urllib3 from 2.5.0 to 2.6.0 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5704](external-secrets/external-secrets#5704) - chore(deps): bump pymdown-extensions from 10.17.2 to 10.18 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5706](external-secrets/external-secrets#5706) - chore(deps): bump alpine from 3.22.2 to 3.23.0 in /e2e by [@dependabot](https://github.com/dependabot)\[bot] in [#5701](external-secrets/external-secrets#5701) - chore(deps): bump golang from `2611181` to `2611181` by [@dependabot](https://github.com/dependabot)\[bot] in [#5721](external-secrets/external-secrets#5721) - chore(deps): bump codecov/codecov-action from 5.5.1 to 5.5.2 by [@dependabot](https://github.com/dependabot)\[bot] in [#5725](external-secrets/external-secrets#5725) - chore(deps): bump urllib3 from 2.6.0 to 2.6.2 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5730](external-secrets/external-secrets#5730) - chore(deps): bump github/codeql-action from 4.31.7 to 4.31.8 by [@dependabot](https://github.com/dependabot)\[bot] in [#5726](external-secrets/external-secrets#5726) - chore(deps): bump anchore/sbom-action from 0.20.10 to 0.20.11 by [@dependabot](https://github.com/dependabot)\[bot] in [#5724](external-secrets/external-secrets#5724) - chore(deps): bump tornado from 6.5.2 to 6.5.3 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5732](external-secrets/external-secrets#5732) - chore(deps): bump ubi9/ubi from `75937d9` to `d4feb57` by [@dependabot](https://github.com/dependabot)\[bot] in [#5722](external-secrets/external-secrets#5722) - chore(deps): bump golang from `5117d68` to `09f53de` in /e2e by [@dependabot](https://github.com/dependabot)\[bot] in [#5729](external-secrets/external-secrets#5729) - chore(deps): bump alpine from `4b7ce07` to `51183f2` by [@dependabot](https://github.com/dependabot)\[bot] in [#5694](external-secrets/external-secrets#5694) - chore(deps): bump hashicorp/setup-terraform from [`712b439`](external-secrets/external-secrets@712b439) to [`071811a`](external-secrets/external-secrets@071811a) by [@dependabot](https://github.com/dependabot)\[bot] in [#5727](external-secrets/external-secrets#5727) - chore(deps): bump pymdown-extensions from 10.18 to 10.19.1 in /hack/api-docs by [@dependabot](https://github.com/dependabot)\[bot] in [#5731](external-secrets/external-secrets#5731) - chore(deps): bump step-security/harden-runner from 2.13.3 to 2.14.0 by [@dependabot](https://github.com/dependabot)\[bot] in [#5728](external-secrets/external-secrets#5728) - chore(deps): bump actions/cache from 4.3.0 to 5.0.1 by [@dependabot](https://github.com/dependabot)\[bot] in [#5723](external-secrets/external-secrets#5723) #### New Contributors - [@iypetrov](https://github.com/iypetrov) made their first contribution in [#5685](external-secrets/external-secrets#5685) - [@tokiwong](https://github.com/tokiwong) made their first contribution in [#5708](external-secrets/external-secrets#5708) - [@Gabryel8818](https://github.com/Gabryel8818) made their first contribution in [#5652](external-secrets/external-secrets#5652) - [@mikesellitto](https://github.com/mikesellitto) made their first contribution in [#5475](external-secrets/external-secrets#5475) - [@lostick](https://github.com/lostick) made their first contribution in [#5736](external-secrets/external-secrets#5736) - [@cmoscofian](https://github.com/cmoscofian) made their first contribution in [#5622](external-secrets/external-secrets#5622) - [@headcr4sh](https://github.com/headcr4sh) made their first contribution in [#5745](external-secrets/external-secrets#5745) - [@tosih](https://github.com/tosih) made their first contribution in [#5610](external-secrets/external-secrets#5610) **Full Changelog**: <external-secrets/external-secrets@v1.1.1...v1.2.0> </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://gitea.alexlebens.dev/alexlebens/infrastructure/pulls/2737 Co-authored-by: Renovate Bot <renovate-bot@alexlebens.net> Co-committed-by: Renovate Bot <renovate-bot@alexlebens.net>

github-project-automation bot added this to External Secrets Dec 17, 2025

github-actions bot added the component/github-actions label Dec 17, 2025

github-actions bot added the size/xs label Dec 17, 2025

coderabbitai bot reviewed Dec 17, 2025

View reviewed changes

evrardjp force-pushed the add_mapping_for_breaking_changes branch from c72c1ff to 74f91a1 Compare December 17, 2025 10:31

coderabbitai bot reviewed Dec 17, 2025

View reviewed changes

fix(testing)!: Breaking changes should not break ci

cdc5513

Without this, a ! in commit title will break labeler. This is a problem, as it means manually labeling issues. This fixes it by correcting the github action. Signed-off-by: Jean-Philippe Evrard <open-source@a.spamming.party>

evrardjp force-pushed the add_mapping_for_breaking_changes branch from 74f91a1 to cdc5513 Compare December 17, 2025 10:38

evrardjp changed the title ~~fix(testing)!: Breaking changes should not break ci~~ babar is the king of the elephants Dec 17, 2025

evrardjp changed the title ~~babar is the king of the elephants~~ fix(testing)!: Breaking changes should not break ci Dec 17, 2025

Merge branch 'main' into add_mapping_for_breaking_changes

eb761a0

evrardjp changed the title ~~fix(testing)!: Breaking changes should not break ci~~ fix(testing): Breaking changes should not break ci Dec 17, 2025

github-actions bot added area/testing Issues / Pull Requests related to testing kind/bug Categorizes issue or PR as related to a bug. labels Dec 17, 2025

Skarlso approved these changes Dec 17, 2025

View reviewed changes

Skarlso merged commit a79a3b2 into external-secrets:main Dec 17, 2025
36 of 37 checks passed

github-project-automation bot moved this to Done in External Secrets Dec 17, 2025

coderabbitai bot mentioned this pull request Jan 14, 2026

fix: try to fix most of the code scanning issues #5812

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(testing): Breaking changes should not break ci#5739

fix(testing): Breaking changes should not break ci#5739
Skarlso merged 2 commits intoexternal-secrets:mainfrom
evrardjp:add_mapping_for_breaking_changes

evrardjp commented Dec 17, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Dec 17, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Dec 17, 2025

Uh oh!

coderabbitai bot Dec 17, 2025

Uh oh!

sonarqubecloud bot commented Dec 17, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

evrardjp commented Dec 17, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem Statement

Related Issue

Proposed Changes

Format

Checklist

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Dec 17, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Dec 17, 2025

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Dec 17, 2025

Quality Gate passed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

evrardjp commented Dec 17, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 17, 2025 •

edited

Loading