Skip to content

Exclude some unnecessary files from release zip - #515

Merged
ockham merged 1 commit into
trunkfrom
update/create-release-zip-script
Jul 21, 2026
Merged

Exclude some unnecessary files from release zip#515
ockham merged 1 commit into
trunkfrom
update/create-release-zip-script

Conversation

@ockham

@ockham ockham commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

What

Two small additions to bin/create-release-zip.sh to keep unnecessary files out of the distribution zip:

  • Remove assets/src/ from the staged plugin directory (~1.4 MB of uncompiled JS partials and Sass). Nothing references it at runtime — only assets/build is loaded. This also matches ACF's WordPress.org distribution, whose assets/ dir contains only build, images, and inc.
  • Delete .DS_Store files anywhere in the staged tree, as the last step before zipping. These are gitignored, so the script's untracked-files guard (git ls-files --others --exclude-standard) never flags them, but the plain cp -r copies them into the staging dir anyway — the current release zip ships an includes/.DS_Store. The cleanup runs after the vendor/ copy so it sweeps the whole tree.

Why

The current zip (6.9.2) includes both of these; neither is needed by users. Other candidates (source maps, unminified builds, .po files) were deliberately left in — sourcemaps help users debug, and ACF ships .po files too.

Testing

  • bash -n bin/create-release-zip.sh passes.
  • Run bin/create-release-zip.sh on a clean tree and verify the resulting zip contains no assets/src/ directory and no .DS_Store files: unzip -l release/secure-custom-fields.zip | grep -E 'assets/src|DS_Store' should return nothing.

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props bernhard-reiter.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@ockham
ockham merged commit 76cc470 into trunk Jul 21, 2026
19 checks passed
@ockham
ockham deleted the update/create-release-zip-script branch July 21, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant