This repository has been archived by the owner on Aug 13, 2023. It is now read-only.
Remove Lerna #297
Labels
low-priority
Refinement Needed
This is ready for refinement. It shouldn't be worked on until it has been refined by Dev & Test.
simorgh-core-stream
Is your feature request related to a problem? Please describe.
Currently, our recommended setup uses lerna.
npm run install:packages
(runs npm install and lerna bootstrap). The standard test command runslerna run test
as well.These are setups both for local development and also on CI.
This is manageable currently, since currently Psammead is used by a single team and all of the components are used for that team's work. When we shortly will have multiple teams' components within this library, as developers we will be often installing dependencies and running tests that will be unnecessary for our project. This wastes time.
We should remove lerna and have a custom script implementation.
Describe the solution you'd like
We should remove lerna and instead have these npm scripts:
install-named-packages
allows passing in of a list of packages and only installs those locally.test-named-packages
allows passing in a list of packages and only runs the npm test command for those.link-named-packages
allows passing in of a list of packages and npm links those locally.The documentation should be updated to reflect this.
We should also have this functionality:
If we have any library-level tests, e.g ones against storybook, we should run these too.
Describe alternatives you've considered
N/A
Testing notes
[Tester to complete]
Dev insight: Instructions should be clear to follow. We should be able to run the scripts locally as well as on CI.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: