Skip to content

ci(renovate): fix configuration issues#1478

Merged
amilcarlucas merged 1 commit into
masterfrom
renovate_fixes
Apr 9, 2026
Merged

ci(renovate): fix configuration issues#1478
amilcarlucas merged 1 commit into
masterfrom
renovate_fixes

Conversation

@amilcarlucas
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 9, 2026 21:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s Renovate configuration to resolve configuration problems and streamline settings for dependency/vulnerability alerting.

Changes:

  • Removes platform/repositories fields from renovate.json (typically not needed for repo-based configs).
  • Simplifies vulnerabilityAlerts configuration formatting.
  • Removes a Python custom.regex manager that previously matched required_packages = [...] patterns.

Comment thread renovate.json
Comment on lines 127 to 139
"customType": "regex",
"managerFilePatterns": ["/\\.py$/"],
"matchStrings": [
"import (?<depName>matplotlib|lxml|beautifulsoup4|requests)(?:\\s|$)"
],
"datasourceTemplate": "pypi",
"currentValueTemplate": "latest"
},
{
"customType": "regex",
"managerFilePatterns": ["/\\.py$/"],
"matchStrings": ["required_packages = \\[[^\\]]*\"(?<depName>[^\"]+)\""],
"datasourceTemplate": "pypi",
"currentValueTemplate": "latest"
},
{

"customType": "regex",
"managerFilePatterns": ["/\\.bat$/"],
"matchStrings": [
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

Removing the custom.regex manager that matched required_packages = [...] in .py files means Renovate will no longer detect dependencies that are only listed there (e.g., required_packages = ["lxml"] in scripts/generate_pdef.xml_metadata.py). The remaining Python regex managers won’t match this case (pip install uses a variable, and there is no import lxml), so consider re-adding a corrected regex for required_packages (or adjusting the script/config so the dependency is discoverable) to avoid silently losing updates for these packages.

Copilot uses AI. Check for mistakes.
@amilcarlucas amilcarlucas force-pushed the renovate_fixes branch 3 times, most recently from 6100dfc to b0aa130 Compare April 9, 2026 22:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
12052 11339 94% 89% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: fd9b549 by action🐍

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

Test Results

     4 files       4 suites   41m 9s ⏱️
 3 527 tests  3 525 ✅  2 💤 0 ❌
13 900 runs  13 876 ✅ 24 💤 0 ❌

Results for commit b0aa130.

@amilcarlucas amilcarlucas merged commit 5c08529 into master Apr 9, 2026
28 checks passed
@amilcarlucas amilcarlucas deleted the renovate_fixes branch April 9, 2026 22:51
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