Skip to content

refactor: Reduce loop nesting for processSupersedesManagers#37733

Merged
secustor merged 1 commit intorenovatebot:mainfrom
zharinov:refactor/supersedes-manager
Aug 27, 2025
Merged

refactor: Reduce loop nesting for processSupersedesManagers#37733
secustor merged 1 commit intorenovatebot:mainfrom
zharinov:refactor/supersedes-manager

Conversation

@zharinov
Copy link
Copy Markdown
Collaborator

Changes

Use early exists and make nested loops look clear.
Increase tests granularity

Context

Please select one of the below:

  • This closes an existing Issue: #
  • This doesn't close an Issue, but I accept the risk that this PR may be closed if maintainers disagree with its opening or implementation

AI assistance disclosure

Did you use AI tools to create any part of this pull request?

Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.

  • No — I did not use AI for this contribution.
  • Yes — minimal assistance (e.g., IDE autocomplete, small code completions, grammar fixes).
  • Yes — substantive assistance (AI generated non‑trivial portions of code, tests, or documentation).
  • Yes — other (please describe):

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

The public repository:

@zharinov zharinov requested a review from viceice August 27, 2025 14:49
@viceice viceice requested review from rarkins and secustor August 27, 2025 14:54
@secustor secustor added this pull request to the merge queue Aug 27, 2025
Merged via the queue into renovatebot:main with commit 585f1d5 Aug 27, 2025
40 checks passed
@renovate-release
Copy link
Copy Markdown

🎉 This PR is included in version 41.87.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

ducthinh993 pushed a commit to ducthinh993/renovate that referenced this pull request Sep 11, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants