Is the package lint test reliable?
Permalink
I've just tested my package at the Marketplace Linter:
http://www.concrete5.org/marketplace/linter...
And following tests have failed:
- The `has_icon_test` -> The package has an icon in PNG format 185x185px at the root of my package (same level as the controller)
- The `invalid_legacy_namespace_test` -> in a HelpServiceProvider I'v added, which works just fine.
I'm confused, because my package installs fine, the HelpServiceProvider works fine (No namespace errors at all!!)
**So I'm asking myself if the Linter is reliable?**
http://www.concrete5.org/marketplace/linter...
And following tests have failed:
- The `has_icon_test` -> The package has an icon in PNG format 185x185px at the root of my package (same level as the controller)
- The `invalid_legacy_namespace_test` -> in a HelpServiceProvider I'v added, which works just fine.
I'm confused, because my package installs fine, the HelpServiceProvider works fine (No namespace errors at all!!)
**So I'm asking myself if the Linter is reliable?**
Thank you for the quick answer.
1. In the guide it says that the icon size doesn't matter as long it isn't bigger than 200x200px
https://www.concrete5.org/documentation/developers/5.7/packages/dire...
2. I've added my HelpServiceProvider exactly as described here:
https://www.concrete5.org/documentation/developers/5.7/packages/impl...
So, I'm stiil a bit confused, as I developed this package from scratch for 5.7. It never existed in 5.6
1. In the guide it says that the icon size doesn't matter as long it isn't bigger than 200x200px
https://www.concrete5.org/documentation/developers/5.7/packages/dire...
2. I've added my HelpServiceProvider exactly as described here:
https://www.concrete5.org/documentation/developers/5.7/packages/impl...
So, I'm stiil a bit confused, as I developed this package from scratch for 5.7. It never existed in 5.6
On your first point, I suspect you are correct and that the linter has not been updated to match what 5.7 will accept. As the icon will only ever be displayed at the smaller size (see the submitting to the marketplace page), you may as well shrink it to size and save yourself the hassle.
On the second point, that is new functionality and I have not seen any implementation of that come through the PRB so far (sorry, I got the wrong tangent with my answer about helpers, a totally different topic to help). I suspect you may be correct again that the linter is not up-to-date with that.
I suggest you PM Korvin and point him at this thread. He looks after the linter. And/or you could submit the package for review with the second point failing and someone who knows the core inside out like @mnkras will then assess and grant an exception on that test so the review can continue.
On the second point, that is new functionality and I have not seen any implementation of that come through the PRB so far (sorry, I got the wrong tangent with my answer about helpers, a totally different topic to help). I suspect you may be correct again that the linter is not up-to-date with that.
I suggest you PM Korvin and point him at this thread. He looks after the linter. And/or you could submit the package for review with the second point failing and someone who knows the core inside out like @mnkras will then assess and grant an exception on that test so the review can continue.
Ouff! I feel quite relieved now... I already tougt that my skills aren't what I ment them to be ;-)
Thank you again. I'll PM to Korvin tomorrow.
As for the icon, the size of it isn't that important after all.
But for now, I get myself a beer cause I've made my day (Timezone Western Europe/Berlin) .
Wish you a good day & and a beer too at the end.
Thank you again. I'll PM to Korvin tomorrow.
As for the icon, the size of it isn't that important after all.
But for now, I get myself a beer cause I've made my day (Timezone Western Europe/Berlin) .
Wish you a good day & and a beer too at the end.
Update:
Here is the path to the HelpProvider:
/packages/package_handle/src/Concrete/Help/HelpServiceProvider.php
And it looks like this:
Here is the path to the HelpProvider:
/packages/package_handle/src/Concrete/Help/HelpServiceProvider.php
And it looks like this:
namespace Concrete\Package\PackageHandle\Help; use Concrete\Core\Foundation\Service\Provider; class HelpServiceProvider extends Provider { public function register() { //...
The invalid legacy namespace test is a test that was added recently to circumvent a common legacy issue with moving to better namespacing in 5.7.5, it is safe to ignore in cases where it is not legacy usage.
I just ran the linter on my package (a theme) and got this failed test
theme_typography_exists_test
File: fluid960gs/
null
How do I fix/clear this error.
theme_typography_exists_test
File: fluid960gs/
null
How do I fix/clear this error.
The second point is because there are things that still work but are deprecated. Within the package, stuff generally has to be within /src/ (including helpers unless you have a custom autoloader), then helpers need to be registered and finally loaded with core make.
Most of the old Loader::Something stuff from 5.6 still works, but is deprecated. Its your choice for your own use, but not for the marketplace.