Skip to content

fix: treat extra files in batch input folder as warning, not error#552

Open
octo-patch wants to merge 1 commit intoHuanshere:mainfrom
octo-patch:fix/issue-518-extra-input-files-warning-only
Open

fix: treat extra files in batch input folder as warning, not error#552
octo-patch wants to merge 1 commit intoHuanshere:mainfrom
octo-patch:fix/issue-518-extra-input-files-warning-only

Conversation

@octo-patch
Copy link
Copy Markdown

Fixes #518

Problem

When the batch/input folder contains more files than are listed in tasks_setting.xlsx, check_settings() would set all_passed = False and fail the entire batch pre-flight check. This prevented processing the correctly configured tasks even though the extra input files are not actually an error condition.

The panel title also used bold red which is misleading for a non-fatal issue.

Solution

  • Remove the all_passed = False assignment from the extra-files branch so that having untracked files in the input folder is a warning only and does not block batch processing.
  • Change the panel title color from bold red to bold yellow to better reflect the non-fatal severity.

Testing

Placed an extra file in batch/input that is not referenced in tasks_setting.xlsx. The settings check now prints a yellow warning about the extra file but continues and returns True when all configured tasks are valid.

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.

提一个bug

1 participant