ci(renovate): fix configuration issues#1478
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the repository’s Renovate configuration to resolve configuration problems and streamline settings for dependency/vulnerability alerting.
Changes:
- Removes
platform/repositoriesfields fromrenovate.json(typically not needed for repo-based configs). - Simplifies
vulnerabilityAlertsconfiguration formatting. - Removes a Python
custom.regexmanager that previously matchedrequired_packages = [...]patterns.
| "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": [ |
There was a problem hiding this comment.
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.
6100dfc to
b0aa130
Compare
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified FilesNo covered modified files...
|
Test Results 4 files 4 suites 41m 9s ⏱️ Results for commit b0aa130. |
b0aa130 to
fd9b549
Compare
No description provided.