Skip to content

fix(import): centralize ignored filename filtering - #1013

Merged
PaulHax merged 2 commits into
Kitware:mainfrom
PaulHax:import-os-metadata-filter
Nov 7, 2026
Merged

PaulHax merged 2 commits into
Kitware:mainfrom
PaulHax:import-os-metadata-filter

Conversation

@PaulHax

@PaulHax PaulHax commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

macOS resource forks (._*) and Finder .DS_Store files caused import errors, while LICENSE files were ignored only inside ZIP archives.

Skip files named LICENSE (case-insensitive), .DS_Store, or starting with ._ wherever they enter the import queue: local files, dropped folders, URLs, and ZIP members. ZIP members are skipped before decompression. __MACOSX/ holds only ._* files, so the name check covers it. When every selected file is skipped, a notice lists them instead of reporting a load.

@netlify

netlify Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for volview-dev ready!

Name Link
🔨 Latest commit 7707fa4
🔍 Latest deploy log https://app.netlify.com/projects/volview-dev/deploys/6aef4fa3ca8c80f76928d002
😎 Deploy Preview https://deploy-preview-1013--volview-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PaulHax
PaulHax force-pushed the import-os-metadata-filter branch from cf07671 to 709ed3d Compare November 2, 2026 17:24
@PaulHax PaulHax changed the title fix(import): skip macOS resource fork and Finder files fix(import): centralize ignored filename filtering Nov 2, 2026
Skip LICENSE, .DS_Store and ._* files wherever they enter the import
queue, including ZIP members before decompression. Report when every
selected file is skipped.
@PaulHax
PaulHax force-pushed the import-os-metadata-filter branch from 627ba75 to c8e90ac Compare November 7, 2026 15:31
@PaulHax
PaulHax merged commit 849b615 into Kitware:main Nov 7, 2026
7 of 8 checks passed
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.

1 participant