You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Please consider adding some (automated) tests, in particular to ensure compatibility with FreshRSS.
See example Joedmin/xExtension-wallabag-button#1
A little detail is that on the main extensions repo, this extension is generating a warning (due to a limitation of our test tools), which would be nice to get rid of:
------ ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Line third-party/8c042738017d75b31bcbc23d84cfb35d6315f3a7/comics/penny-arcade.php
------ ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
:59 Inner named functions are not supported by PHPStan. Consider refactoring to an anonymous function, class method, or a top-level-defined function. See issue #165 (https://github.com/phpstan/phpstan/issues/165) for more details.
------ ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
The text was updated successfully, but these errors were encountered:
I currently don't have time to implement tests, but will keep that in mind for the future. If anyone is reading this and wants to do a PR for this, I can merge it
Please consider adding some (automated) tests, in particular to ensure compatibility with FreshRSS.
See example
Joedmin/xExtension-wallabag-button#1
A little detail is that on the main extensions repo, this extension is generating a warning (due to a limitation of our test tools), which would be nice to get rid of:
The text was updated successfully, but these errors were encountered: