Skip to content

Ignore .extract-wp-hooks.json & .markdownlintignore config files#1152

Merged
mauteri merged 1 commit intoGatherPress:developfrom
carstingaxion:patch-1
Aug 7, 2025
Merged

Ignore .extract-wp-hooks.json & .markdownlintignore config files#1152
mauteri merged 1 commit intoGatherPress:developfrom
carstingaxion:patch-1

Conversation

@carstingaxion
Copy link
Copy Markdown
Collaborator

@carstingaxion carstingaxion commented Aug 7, 2025 •
edited
Loading

Description of the Change

Second follow-up to #1135

Removes the config files for markdownlint and extract-wp-hooks build processes from the final distributed plugin.

Closes #

How to test the Change

Changelog Entry

Fixed - Removes the config files for markdownlint and extract-wp-hooks build processes from the final distributed plugin.

Credits

Props @carstingaxion

Checklist:

  • I agree to follow this project's Code of Conduct.
  • All new and existing tests pass.

@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Aug 7, 2025

PR Summary

  • File addition to .distignore
    The '.extract-wp-hooks.json' file has been included in the .distignore file. This change means that the aforementioned file won't be distributed in production environments, reducing unnecessary clutter and enhancing the overall performance.
  • File addition to .distignore
    The '.markdownlintignore' file has also been added to the .distignore file. This will prevent the unnecessary inclusion of markdown lint exceptions in the distribution, leading to cleaner releases.

@carstingaxion carstingaxion marked this pull request as ready for review August 7, 2025 12:39
@mauteri mauteri merged commit 6958458 into GatherPress:develop Aug 7, 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

@carstingaxion @mauteri