bito-code-review[bot] commented on PR #42482:
URL: https://github.com/apache/superset/pull/42482#issuecomment-5219304429
<!-- Bito Reply -->
The flagged issue is correct. Adding the scheduled Docker refresh workflow
to the `python` pattern in `scripts/change_detector.py` causes it to be
classified as Python code, triggering a wide range of unnecessary CI jobs
whenever this workflow file is modified.
To resolve this, you should remove the specific workflow file from the
`python` pattern and, if necessary, create a dedicated category for
infrastructure or workflow changes that only triggers essential checks.
### Proposed Fix
In `scripts/change_detector.py`, remove the line adding the workflow to the
`python` pattern:
```python
"python": [
r"^\.github/workflows/.*python",
r"^docker-compose-image-tag\.yml$",
r"^tests/",
```
(Note: The line
`r"^\.github/workflows/scheduled-docker-image-refresh\.yml$",` should be
deleted.)
There are no other comments on this PR to address.
**scripts/change_detector.py**
```
"python": [
r"^\.github/workflows/.*python",
r"^docker-compose-image-tag\.yml$",
r"^tests/",
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
