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

Ensure that the pot file use the same license as the plugin #199

Merged
merged 3 commits into from
Dec 13, 2019

Conversation

ponsfrilus
Copy link
Contributor

It's green :)
image
image

This PR now includes a test case that ensure that the licensing information in the pot file is taken from the plugin file header if exists. Fixes #198.

ponsfrilus and others added 3 commits December 11, 2019 16:14
The doc says: 
```
If a plugin or theme specifies a license in their main plugin file or stylesheet, the comment looks like
this: 
Copyright (C) 2018 Example Plugin Author
This file is distributed under the GPLv2.
```
but it dosen't seems to work for plugins. It looks like the "License" param in get_file_headers was missing.
* Only if the license file header is defined
* Tests updated
@schlessera schlessera merged commit 2804c52 into wp-cli:master Dec 13, 2019
@schlessera schlessera added this to the 2.2.2 milestone Dec 13, 2019
@schlessera
Copy link
Member

Thanks for the PR, @ponsfrilus !

schlessera added a commit that referenced this pull request Jan 6, 2022
Ensure that the pot file use the same license as the plugin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants