Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add required PHP header #487

Merged
merged 2 commits into from
Jan 16, 2025
Merged

Add required PHP header #487

merged 2 commits into from
Jan 16, 2025

Conversation

mikkamp
Copy link
Contributor

@mikkamp mikkamp commented Jan 16, 2025

Changes proposed in this Pull Request:

This PR adds the required PHP header in the main plugin file. It also sets the PHP version for composer packages. Note that this is only for dev packages, since we don't bundle any packages in the release.

Closes #486

Note

Since this doesn't have any effect on the release bundle I'll go ahead and merge without a review.

Changelog entry

  • Tweak - Add required PHP header.

@mikkamp mikkamp self-assigned this Jan 16, 2025
@github-actions github-actions bot added the changelog: tweak Small change, that isn't actually very important. label Jan 16, 2025
@mikkamp mikkamp merged commit 86cb647 into trunk Jan 16, 2025
8 checks passed
@mikkamp mikkamp deleted the tweak/486-add-php-header branch January 16, 2025 09:23
@ianlin ianlin mentioned this pull request Jan 21, 2025
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: tweak Small change, that isn't actually very important.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

QIT validation check shows missing PHP header
1 participant