ort
ort copied to clipboard
oss-review-toolkit
Reformat README
I'm ok to take it into use, but we should ASAP implement checks to enforce this (for already migrated files).
Like I mentioned in the core dev meeting, the problem is that there is no rule for that. There are two markdownlint plugins which we might use, but both of them don't seem to be actively maintained and I don't know how well they work: https://www.npmjs.com/package/markdownlint-rule-max-one-sentence-per-line https://www.npmjs.com/package/sentences-per-line
I could give it a try anyway.
What's blocking to merge this @mnonnenmacher?
I wasn't sure if you are stil ok with the merge in case it does not work to use one of the rules mentioned above, and I haven't found the time to test it yet.
I wasn't sure if you are stil ok with the merge
I'm still fine. Let's merge, and if the feel comfortable with the style, come up with automated checks and roll it out further. And if we are not fine, this change is small enough to revert (even manually).