glynnbird opened a new pull request #53: bring follow library under Nano's wing
URL: https://github.com/apache/couchdb-nano/pull/53
 
 
   ## Overview
   
   Currently Nano uses a the [follow](https://www.npmjs.com/package/follow) 
library as a dependency to handle changes feed subscription. Unfortunately, 
`follow` is not maintained. The nice people at Cloudant have taken the code and 
fixed it up and published it 
[here](https://github.com/cloudant-labs/cloudant-follow).  This 
[PR](https://github.com/apache/couchdb-nano/pull/51) proposes to depend on the  
cloudant-follow fork, but I am proposing something different:
   
   1) Bring the follow codebase into the Nano project (it is published under an 
Apache-2.0 license)
   2) Remove follow as a dependency
   3) In the future refactor the follow code to be more in keeping with the 
   
   ## Testing recommendations
   
   This should be a drop-in replacement of the previous version. The only 
difference is that the `follow` code lives in this repo and not as a dependency.
   
   I've brought in the `follow` tests too. Unfortunately they rely on 
CouchDB1.6 so I've altered the testing sequence to run our core tests with 
CouchDB2 then switch over to CouchDB1.6 for the follow tests.
   
   See `npm run coretests` & `npm run followtests` in package.json.
   
   ## GitHub issue number
   
   <!-- If this is a significant change, please file a separate issue at:
        https://github.com/apache/couchdb-nano/issues
        and include the number here and in commit message(s) using
        syntax like "Fixes #472" or "Fixes apache/couchdb#472".  -->
   
   ## Related Pull Requests
   
   See issue #28 & pull #51 
   
   ## Checklist
   
   - [x ] Code is written and works correctly;
   - [x] Changes are covered by tests;
   - [x ] Documentation reflects the changes;
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to