Skip to content
This repository has been archived by the owner on Jun 23, 2023. It is now read-only.

Links to Scrutinizer and to Packagist are wrong #105

Closed
mondrake opened this issue Nov 4, 2017 · 6 comments
Closed

Links to Scrutinizer and to Packagist are wrong #105

mondrake opened this issue Nov 4, 2017 · 6 comments

Comments

@mondrake
Copy link
Contributor

mondrake commented Nov 4, 2017

The pel/pel package on Packagist refers to another project, and the badge on the readme is wrong. So it looks like in composer you still need to require lsolesel/pel.

Also, the badge linking to Scrutinizer leads to a 404 page.

@weberhofer
Copy link
Collaborator

Sorry, yes. I hope Lars acceps the request to add the project to Scrutinizer soon. I can't.

@lsolesen
Copy link
Collaborator

lsolesen commented Nov 4, 2017

@weberhofer Sorry, I set your role wrong. It should be fixed now, so you are and admin of the pel organization.

weberhofer added a commit that referenced this issue Nov 5, 2017
* Enabled code rating
* Fixes for #105 

* Fixed version

* override tests

* Re-wrote config

* Added phpunit

* Added forgotten build section

* Try to fix format

* Fixed command

* Enable code rating and duplication checks

* php-clover->clover
@weberhofer
Copy link
Collaborator

Thanks @lsolesen . Hope the badges are now ok again.

@weberhofer
Copy link
Collaborator

@lsolesen how did you get the coverage badge worgin on the former project?

@weberhofer weberhofer reopened this Nov 5, 2017
@weberhofer
Copy link
Collaborator

It might take a while. On the scruntizer page it's already visible.

@mondrake
Copy link
Contributor Author

mondrake commented Nov 8, 2017

@weberhofer Scrutinizer badges are OK, but it seems that the Packagist are still wrong? The pel/pel package on Packagist is for the PHP Extended Library.

Will post a PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants