-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reorganize READMEs in the monorepo #460
Comments
From what I can say, they look good (I see only one line related to roolup for removing). The README on the main level says about Miew as about the tool (lib + app) where lib level considers Miew only as a library (only lib), There are few notes:
I think in this list we have to remove only rollup? |
Re: banner. I think it is enough to have the only banner at the top of the repository. Re: rollup. I don't even think we should remove rollup here. It is perfectly valid and is still used. The fact that we stopped using it for our builds doesn't mean that our users must do the same. Re: scope. My primary concern was a surprise I got when I saw two readme's - I want to avoid such surprise for our users in the future. So, we must clearly separate the contents and hyperlink the files. We should also add a missing readme in the |
and in general, do we need to mention all tools users can use with the package? Is it not obvious? Just question, I don't really know such details.
so |
Here is the plan as a result of the discussion:
|
We have two "main"
README.md
files in this monorepo, at the root and in thelib/
package. Theapp/
package doesn't have any. Both existing files have badges which is a bit misleading (but perhaps it is fine) --- and have led to fixing the badges twice, in #458 and #459.It is normal to have many readmes in a repo, e.g. one root file and one per package (and even more). It should be clear from their content that there exists a hierarchy. We shouldn't repeat ourselves unless it is necessary.
Let's revisit this and decide on the number and the content of readmes.
The text was updated successfully, but these errors were encountered: