Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

You can preview this rule here (updated a few minutes after each push).

Review

A dedicated reviewer checked the rule description successfully for:

  • logical errors and incorrect information
  • information gaps and missing content
  • text style and tone
  • PR summary and labels follow the guidelines

@denis-troller denis-troller changed the title Create rule S8043 Create rule S8043: DML operations in catch blocks can mask original exceptions Sep 29, 2025
Copy link

Quality Gate passed Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link

Quality Gate passed Quality Gate passed for 'rspec-tools'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link

CI Feedback 🧐

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: build-and-deploy

Failed stage: Deploy 🚀 [❌]

Failed test name: ""

Failure summary:

The GitHub Pages deployment failed during the push step due to a non-fast-forward update:
- Git
reported: cannot lock ref 'refs/heads/gh-pages': is at 24ccc02f52a2b073cb816af70861cf03f2371677 but
expected 6bc87b24b9f08b74b3e06a7b66be5d45fccc761b (lines 838–839).
- This indicates that the remote
gh-pages branch was updated by another process/commit between the action’s fetch and push, so the
forced push could not lock the ref.
- As a result, the deploy step failed with exit code 1, and the
action terminated.

Relevant error logs:
1:  Runner name: 'github-ubuntu-latest-s_1b033624bd22'
2:  Runner group name: 'github'
...

130:  ##[endgroup]
131:  npm warn deprecated [email protected]: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
132:  npm warn deprecated [email protected]: Glob versions prior to v9 are no longer supported
133:  npm warn deprecated @material-ui/[email protected]: Material UI v4 doesn't receive active development since September 2021. See the guide https://mui.com/material-ui/migration/migration-v4/ to upgrade to v5.
134:  npm warn deprecated @material-ui/[email protected]: Material UI v4 doesn't receive active development since September 2021. See the guide https://mui.com/material-ui/migration/migration-v4/ to upgrade to v5.
135:  npm warn deprecated @material-ui/[email protected]: Material UI v4 doesn't receive active development since September 2021. See the guide https://mui.com/material-ui/migration/migration-v4/ to upgrade to v5.
136:  added 412 packages, and audited 413 packages in 9s
137:  63 packages are looking for funding
138:  run `npm fund` for details
139:  found 0 vulnerabilities
140:  > [email protected] predeploy
141:  > npm run prepare-rules && npm run build
142:  > [email protected] prepare-rules
143:  > ts-node -P tsconfig-rules.json ./src/deployment/index.ts
144:  When parsing the JSON file /home/runner/work/rspec/rspec/frontend/incomplete-rules-repo/rules/S7472/python/metadata.json
145:  {"level":"error","message":"Failed to process PR (https://github.com/SonarSource/rspec/pull/4914), it will be skipped (SyntaxError: Expected double-quoted property name in JSON at position 493)","source":"pullRequestIndexing.ts"}
146:  {"level":"error","message":"No languages found for rule S7472, at least 1 is required","source":"searchIndex.ts"}
147:  > [email protected] build
...

260:  * [new branch]            feat/smarini/BUILD-6088-add-security-md -> origin/feat/smarini/BUILD-6088-add-security-md
261:  * [new branch]            feat/sonartech/BUILD-8875-changeRunnersNames -> origin/feat/sonartech/BUILD-8875-changeRunnersNames
262:  * [new branch]            feature/education       -> origin/feature/education
263:  * [new branch]            feature/education-backup -> origin/feature/education-backup
264:  * [new branch]            feature/education-new-checks -> origin/feature/education-new-checks
265:  * [new branch]            feature/education-new-checks-rebased -> origin/feature/education-new-checks-rebased
266:  * [new branch]            feature/education-updated -> origin/feature/education-updated
267:  * [new branch]            fix-cloudformation-s6295 -> origin/fix-cloudformation-s6295
268:  * [new branch]            fix-uncatched-promise   -> origin/fix-uncatched-promise
269:  * [new branch]            fix/re-bot/ownership    -> origin/fix/re-bot/ownership
270:  * [new branch]            fix/typo/S5131          -> origin/fix/typo/S5131
271:  * [new branch]            ft/cleaning             -> origin/ft/cleaning
272:  * [new branch]            ft/common3              -> origin/ft/common3
273:  * [new branch]            ft/cppcoreguidelines    -> origin/ft/cppcoreguidelines
274:  * [new branch]            ft/dirty-security-apply -> origin/ft/dirty-security-apply
275:  * [new branch]            ft/fix-error-login      -> origin/ft/fix-error-login
276:  * [new branch]            ft/moreupdate           -> origin/ft/moreupdate
...

824:  [command]/usr/bin/git status --porcelain
825:  M rules/S8045/apex-description.html
826:  M rules/S8045/apex-metadata.json
827:  M rules/S8045/default-description.html
828:  M rules/S8045/default-metadata.json
829:  M rules/rule-index-aggregates.json
830:  M rules/rule-index-store.json
831:  M rules/rule-index.json
832:  [command]/usr/bin/git add --all .
833:  [command]/usr/bin/git checkout -b github-pages-deploy-action/ho3dmjwm2
834:  Switched to a new branch 'github-pages-deploy-action/ho3dmjwm2'
835:  [command]/usr/bin/git commit -m Deploying to gh-pages from  @ 38635fd75ef4d21cea8aa79f6e1663c13c2a5af3 🚀 --quiet --no-verify
836:  [command]/usr/bin/git push --force ***github.com/SonarSource/rspec.git github-pages-deploy-action/ho3dmjwm2:gh-pages
837:  To https://github.com/SonarSource/rspec.git
838:  ! [remote rejected]       github-pages-deploy-action/ho3dmjwm2 -> gh-pages (cannot lock ref 'refs/heads/gh-pages': is at 24ccc02f52a2b073cb816af70861cf03f2371677 but expected 6bc87b24b9f08b74b3e06a7b66be5d45fccc761b)
839:  error: failed to push some refs to 'https://github.com/SonarSource/rspec.git'
840:  Running post deployment cleanup jobs… 🗑️
841:  [command]/usr/bin/git worktree remove github-pages-deploy-action-temp-deployment-folder --force
842:  ##[error]The deploy step encountered an error: The process '/usr/bin/git' failed with exit code 1 ❌
843:  Deployment failed! ❌
844:  Post job cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant