Skip to content
This repository has been archived by the owner on Sep 18, 2018. It is now read-only.

Fix licence link #181

Merged
merged 1 commit into from
Aug 16, 2016
Merged

Fix licence link #181

merged 1 commit into from
Aug 16, 2016

Conversation

mjenczmyk
Copy link

@mjenczmyk mjenczmyk commented Aug 16, 2016

I've changed licence icon to display GPL licence type and redirect on click to COPYING file.

What was changed is setting proper licence in setup.py, so licence link button will be displayed according to its content.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage remained the same at 94.628% when pulling fbc552c on mjenczmyk:fix-licence-link into 6784172 on ClearcodeHQ:master.

@mmaslowskicc
Copy link
Collaborator

Can we fix it instead by listing the appropriate license in setup.py so shields.io can get this from PyPI?

@mjenczmyk
Copy link
Author

I've done as you @mmaslowskicc suggested.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage remained the same at 94.628% when pulling d17b261 on mjenczmyk:fix-licence-link into 6784172 on ClearcodeHQ:master.

@@ -94,6 +95,7 @@ def read(fname):
'Topic :: Software Development :: Quality Assurance',
'Topic :: Software Development :: Testing',
'Topic :: Utilities',
'License :: OSI Approved :: GNU General Public License v3 (GPLv3)',
Copy link
Collaborator

Choose a reason for hiding this comment

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

License :: OSI Approved :: GNU Lesser General Public License v3 or later (LGPLv3+)

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage remained the same at 94.628% when pulling 4a1e166 on mjenczmyk:fix-licence-link into 6784172 on ClearcodeHQ:master.

@mmaslowskicc
Copy link
Collaborator

Looks ok.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage decreased (-0.03%) to 94.601% when pulling 727f9ef on mjenczmyk:fix-licence-link into 6784172 on ClearcodeHQ:master.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage decreased (-0.04%) to 94.592% when pulling 82121c8 on mjenczmyk:fix-licence-link into 6784172 on ClearcodeHQ:master.

@fizyk fizyk merged commit 8a8ae41 into ClearcodeHQ:master Aug 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants