Skip to content

Skip non-directory entries in cleanUp results folder#8

Open
SeaCelo wants to merge 1 commit intoOSeMOSYS:masterfrom
SeaCelo:fix-cleanup-dsstore
Open

Skip non-directory entries in cleanUp results folder#8
SeaCelo wants to merge 1 commit intoOSeMOSYS:masterfrom
SeaCelo:fix-cleanup-dsstore

Conversation

@SeaCelo
Copy link

@SeaCelo SeaCelo commented Feb 3, 2026

Fixes #7

  • Skip non-directory entries when iterating over the results folder during cleanUp(). Not an issue on Windows but a good patch to prevent errors on any future platforms.

Why

  • On macOS, Finder creates .DS_Store files in folders.
  • cleanUp() assumes every entry in res/ is a directory and crashes with NotADirectoryError when it hits .DS_Store.
  • This guard avoids the error without changing any behavior on other platforms.

Testing

  • Clicked CLEAN UP on macOS; no 500 error.

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.

CLEAN UP fails on macOS due to .DS_Store in res folder

1 participant