Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

chore: monorepo separating interfaces and compliance tests #97

Conversation

vasco-santos
Copy link
Member

@vasco-santos vasco-santos commented May 28, 2021

This PR makes libp2p-interfaces a monorepo with its interfaces and compliance tests in two different modules.

Closes #58

BREAKING CHANGE: the tests now live in the libp2p-interfaces-compliance-tests module

@vasco-santos vasco-santos force-pushed the chore/monorepo-separating-interfaces-and-compliance-tests branch 16 times, most recently from f0700d7 to a3e18cf Compare May 28, 2021 17:09
@vasco-santos vasco-santos force-pushed the chore/monorepo-separating-interfaces-and-compliance-tests branch from a3e18cf to abbea77 Compare June 24, 2021 14:23
"it-goodbye": "^3.0.0",
"it-pair": "^1.0.0",
"it-pipe": "^1.1.0",
"libp2p-interfaces": "^0.11.0",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@achingbrain how do you handle dependency updates of adjacent modules?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean when sibling versions change within a monorepo? Lerna takes care of all that.

@vasco-santos vasco-santos marked this pull request as ready for review June 24, 2021 14:34
@vasco-santos vasco-santos requested a review from achingbrain June 24, 2021 14:34
@achingbrain achingbrain force-pushed the chore/monorepo-separating-interfaces-and-compliance-tests branch 4 times, most recently from bf7b63e to 3cb0ed9 Compare July 7, 2021 11:47
@achingbrain achingbrain force-pushed the chore/monorepo-separating-interfaces-and-compliance-tests branch from 3cb0ed9 to 9f4e3c9 Compare July 7, 2021 11:52
@vasco-santos vasco-santos merged commit 946348f into master Jul 7, 2021
@vasco-santos vasco-santos deleted the chore/monorepo-separating-interfaces-and-compliance-tests branch July 7, 2021 12:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split the tests or interfaces out?
2 participants