Comments (8)
I just realized that this nukes all of the git history which I hate. I'll have to look into fixing that.
from web.
Do we not want some of these to be together in some way?
Like doesn't putting all the Rollup plugins in one place make them easier to keep up to date?
@web/storybook-builder
and @web/storybook-framework-web-components
are partners anyways?
I can't speak to the Dev Server tools you're moving, but @web/test-runner-visual-regression
is a pretty important piece of the testing ecosystem, doesn't it make better sense for it to continue living with the rest of the Test Runner tools?
from web.
Like doesn't putting all the Rollup plugins in one place make them easier to keep up to date?
I've personally found it harder to maintain packages as they are colocated in a monorepo so this was supposed to help with that. The monorepo kind of mandates that there are specific TypeScript settings and we've got some custom scripts to keep that all in check. Having these just all in their own repos allows us to use off the shelve tooling wihtout all the script glue.
@web/storybook-builder
and@web/storybook-framework-web-components
are partners anyways?
Yeah I feel like these could then be merged into a single repo. I'm not too familiar so I'm not sure.
I can't speak to the Dev Server tools you're moving, but
@web/test-runner-visual-regression
is a pretty important piece of the testing ecosystem, doesn't it make better sense for it to continue living with the rest of the Test Runner tools?
My thinking is that modernweb-dev/web
should pretty much just be test-runner
and dev-server
and everything else should live outside that repo. Not all projects need visual regression testing and so I think of it as a plugin rather than a core part of the test runner.
Other alternatives to doing all this would be to stop using npm workspaces and the custom scripts to also simplify the monorepo. Treating the monorepo as just a collection of independent packages but I worry that the API layer will be broken quite often as it's happened in the monorepo as it stands today.
from web.
In an effort to lower the maintenance burden of the Modern Web projects we've decided to move certain packages out of the monorepo into their own repositories.
Can you please elaborate how splitting the monorepo would lower the maintenance?
Sure there are pros and cons of monorepos vs multiple repose, but curious about your personal experience here and what brought you to this decision?
from web.
In an effort to lower the maintenance burden of the Modern Web projects we've decided to move certain packages out of the monorepo into their own repositories.
Can you please elaborate how splitting the monorepo would lower the maintenance? Sure there are pros and cons of monorepos vs multiple repose, but curious about your personal experience here and what brought you to this decision?
It's a lot easier to make changes in isolation. With the current setup you have to make huge sweeping changes like this: #2600.
The current setup also kind of assumes that every package is using the same technology, as different packages have grown at different paces we've had to add more and more custom scripts to try to mange these. TypeScript with it's different tsconfigs, some projects in ESM and some in CommonJS, etc..
from web.
If we end up reverting this decision I'd like someone to review and approve #2600 in its place.
from web.
Generally speaking, from my personal experience, having a monorepo makes sense if it's the same team maintaining it and everyone has knowledge about most of the packages within it. On the other hand, whether the packages are related to each other logically is not that important.
Also it should be limited in size, basically you can't just add new packages to it forever, so you need to draw a line. Maybe we reached that point.
In this specific case it might be hard to make changes like #2600, because it requires deep knowledge of some packages (e.g. storybook ones) where the logic and requirements for ESM might be different. On top of that the lack of tests makes it hard to verify if such changes are fine, and more packages without automated tests means more burden on the author of such PRs in case things go wrong. I guess this is what cause you a lot of challenges. Same for me as a reviewer: I honestly fear reviewing it as I don't have knoweldge about many of the packages and not sure how to reliably test such changes and sigh them off. Maybe in we can just release it as alpha/beta and ask the community to help with testing?
Having that said, I think it will be hard to split the repository. Because it's not just code, it's also the website with docs behind it, so you'll need to extract it into another repo too, but still central with the entire site content, right? Which will add more maintenance costs for it. It's not clear to me if such trade-off is worth it in the long run. We don't have changes like #2600 often, it's a bit of a special case.
from web.
Because it's not just code, it's also the website with docs behind it, so you'll need to extract it into another repo too, but still central with the entire site content, right?
I don't know about this. As far as I can tell there's nothing that ties packages to the documentation so it shouldn't matter if the actual code is in the repo or not.
from web.
Related Issues (20)
- Issues Implementing @web-test-runner in Docker Environment
- Dooplay theme
- Storybook higher than 7.5.3 does not export globals in @storybook/preview/globals
- Running fails with "cannot find module"
- test-runner-core always emits ansi codes to clear progress bars, even if there aren't progress bars
- Error: ENOENT: no such file or directory, stat '/test-mocks/prism.js'
- [rollup-plugin-html] No clearly documented / easy way to ignore modules
- Use normal `chai` package over `@esm-bundle/chai`
- Web Test Runner terminates randomly without logs HOT 5
- Including/excluding files from coverage based on source path HOT 2
- Tree shaken content shows up as tested in code coverage
- [dev-server-rollup] TypeError: Cannot destructure property 'skipSelf' of 'options' as it is undefined.
- [dev-server] [CVE-2023-42282] Update "ip" package to >=2.0.0 HOT 1
- Error when executing test "Failed to fetch dynamically imported module" HOT 5
- Used Puppeteer version is deprecated HOT 1
- [test-runner] Specifying the root directory of non-script assets HOT 1
- [dev-server-storybook]: distribution filenames with problematic `--` as the first characters HOT 1
- [Server] Mutating source files resulting in them being invalid
- Web development
- Getting websocket error with @web/test-runner-visual-regression HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from web.