Skip to content

Don't include the .wordpress-org directory in the released versions of the plugin#1104

Merged
mauteri merged 2 commits intoGatherPress:developfrom
dd32:patch-1
May 21, 2025
Merged

Don't include the .wordpress-org directory in the released versions of the plugin#1104
mauteri merged 2 commits intoGatherPress:developfrom
dd32:patch-1

Conversation

@dd32
Copy link
Copy Markdown
Contributor

@dd32 dd32 commented May 21, 2025

Description of the Change

While looking at https://plugins.trac.wordpress.org/browser/gatherpress/#tags/0.32.2 and https://plugins.trac.wordpress.org/browser/gatherpress/trunk I noticed that you've included the .wordpress-org directory in the released versions of the plugin, although this is intended to be the assets/ folder only.

See https://github.com/10up/action-wordpress-plugin-deploy?tab=readme-ov-file#excluding-files-from-deployment

How to test the Change

Changelog Entry

Fixed - smaller release ZIPs as we no longer ship the screenshots as part of the release.

Credits

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented May 21, 2025

PR Summary

  • Inclusion of '.wordpress-org' in the '.distignore' file
    This change ensures that the '.wordpress-org' file gets ignored during the distribution process, most likely to avoid unintended side effects or to keep the distribution package lean. Any changes or updates to this file will not be included in the final product release.

@dd32
Copy link
Copy Markdown
Contributor Author

dd32 commented May 21, 2025 •
edited
Loading

Additionally, the docs probably are best not shipped as well.
https://plugins.trac.wordpress.org/browser/gatherpress/tags/0.32.2/docs

Excluding these two folders shaves 11.6MB out of the installed files:

$ du -sh docs .wordpress-org/
7.7M	docs
3.9M	.wordpress-org/

@mauteri
Copy link
Copy Markdown
Contributor

mauteri commented May 21, 2025

Thanks @dd32! Approved and merging.

@mauteri mauteri merged commit 8e89189 into GatherPress:develop May 21, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mauteri mauteri mauteri approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dd32 @mauteri