You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
PR is unable to merge if only .md files are edited, because unittest and analyze_workflow are ignoring those files. But merging changes require all checks to pass which is not happening, because 2 of 3 are not even started.
Proposed fix:
Delate from workflows those lines
paths-ignore:
- '**.md'
Logs and external information
Reproduction script
Make a PR with changes only to .md file.
On what platform(s) does it occur?
Not Applicable
Issue Severity
None
The text was updated successfully, but these errors were encountered:
But agree with complete path-filtering removal. This will keep CI simple. **/*.md doesn't look sophisticated and creating precise filter might be problematic. You may think that only **/*.py changes should be unit-tested but what about yaml templates - they might be used in unit-tests also.
Summary
PR is unable to merge if only
.md
files are edited, because unittest and analyze_workflow are ignoring those files. But merging changes require all checks to pass which is not happening, because 2 of 3 are not even started.Proposed fix:
Delate from workflows those lines
Logs and external information
Reproduction script
Make a PR with changes only to
.md
file.On what platform(s) does it occur?
Not Applicable
Issue Severity
None
The text was updated successfully, but these errors were encountered: