Hi Zamly and vanni, I have reviewed the Feedly connector[1] please go through the comments below.
- Parameter descriptions are missing. - Copy rights are missing. - Remove the unwanted SOP in test. - Remove unwanted comments. - Change the pom to run without skip tests(Now need to give mvn clean install -DskipTests and then mvn test) - The Topics resource is no more working in the feedly api itself now so can ignore it. - *feedly_authentication *resource is not used so can be removed from code. Once done with this changes please give the pull request to [2]. Find the test result [3] (Topics resource excluded) all the test are working fine. Thank you.. -- *S.Elilmatha* Associate Software Engineer, WSO2 Inc.; http://wso2.com lean.enterprise.middleware Mobile 0779842221. [1] https://github.com/zamly/feedlyESBconnector [2] https://github.com/wso2/esb-connectors [3]
FeedlyReviewResults.pdf
Description: Adobe PDF document
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
