search

cn-tools / cntools_FreshRssExtensions

This repository contains my unofficial FreshRSS extensions
MIT License
50 stars 11 forks source link

Add automated tests #36

Open Alkarex opened 1 week ago

Alkarex commented 1 week ago

Please consider adding some (automated) tests, in particular to ensure compatibility with FreshRSS. See example https://github.com/Joedmin/xExtension-wallabag-button/pull/1

On the main extensions repo, this extension is generating a warning, which would be nice to get rid of:

------ ------------------------------------------------------------------------------------------- 
  Line   third-party/003df625d41b62458ad6771961c1bddc4fe76d3b/xExtension-SendToMyJD2/extension.php  
 ------ ------------------------------------------------------------------------------------------- 
  :305   Undefined variable: $lMD                                                                   
 ------ -------------------------------------------------------------------------------------------