Skip to content

Remove redundant license check test case#916

Merged
swissspidy merged 1 commit into
trunkfrom
license-check-tests
Apr 22, 2025
Merged

Remove redundant license check test case#916
swissspidy merged 1 commit into
trunkfrom
license-check-tests

Conversation

@ernilambar
Copy link
Copy Markdown
Member

@ernilambar ernilambar commented Apr 22, 2025

@ernilambar ernilambar marked this pull request as ready for review April 22, 2025 05:13
@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.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: ernilambar <rabmalin@git.wordpress.org>

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

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (3)
  • tests/phpunit/testdata/plugins/test-plugin-plugin-readme-mpl2-license-without-errors/load.php: Language not supported
  • tests/phpunit/testdata/plugins/test-plugin-plugin-readme-mpl2-license-without-errors/readme.txt: Language not supported
  • tests/phpunit/tests/Checker/Checks/Plugin_Readme_Check_Tests.php: Language not supported

@ernilambar ernilambar changed the title Update license check tests Remove redundant license check test case Apr 22, 2025
@swissspidy swissspidy merged commit f2cca4f into trunk Apr 22, 2025
27 of 28 checks passed
@swissspidy swissspidy deleted the license-check-tests branch April 22, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants