Github user eiri commented on the pull request:
https://github.com/apache/couchdb-couch/commit/78f575ec2cac823605561f1b38e0761d8170ac5f#commitcomment-17481247
@janl
Travis was throwing those errors because this change affects two repos -
#161 and
[couchdb-couch-mrview#44](https://github.com/apache/couchdb-couch-mrview/pull/44),
but when travis testing over merge it is pointing only to a new commit in a
current repo and then running test for all the "old" commits in all repos.
In this case that meant that the test was run over updated version of
`couchdb-couch`, but old version of `couchdb-couch-mrview`. When mrview's
change was merged, i.e. when both changes got tested together, the travis test
passed - https://travis-ci.org/apache/couchdb-couch-mrview/builds/126439715
Unfortunately in its current setup travis will always fail when changes
affect more than one repository and have cyclic dependencies. I've merged this
PR because it was reviewed, tested and approved by multiple people and I knew
that I needed both repos tested together to get travis to pass. I understand
that this is not ideal situation and I'd love to improve it, but don't really
know how.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
