Temporary remove --check docs until 1.16 is released to avoid confusion#5671
Temporary remove --check docs until 1.16 is released to avoid confusion#5671j-f1 merged 1 commit intoprettier:masterfrom kachkaev:temporary-hide-check-option
Conversation
How would we be able to fix bugs in the website then? Would we have to commit to both branches? |
|
There must be various ways of doing so I believe. E.g. there can be a With a |
|
We could use versioned docs to achieve this goal, see here for example. (We don't need to add past versions, stable and next should be enough.) |
|
I would totally agree that a separate branch for each major version + dev/canary is a very good idea. I would even go on step further and say every merge you do on master should be auto-released - e.g. you merge a bugfix => get's released. canary would probably be too noisy for that => so merge without release and schedule a release every night. I think something like this would be the ideal release system. |
|
@daKmoR Installing from GitHub is already supported, which means you can simply |
|
@ikatyang if master plays the role of I'm more in favour of If Here is an example of a different approach: https://github.com/containous/traefik. They use One way or another, it would be great to see the development and bugfix work to be split between branches to avoid reverting upcoming docs or keeping feature pull requests open till the last minute. The new workflow will also make patch releases much easier. Imagine someone discovering an urgent bug tomorrow, which would require releasing Meanwhile, I'll prepare a PR that reverts this PR that reverts docs in PR #5629 that confused @daKmoR (and many others) 😅 |
While it sounds like a good idea, our website is currently deployed from master via Netlify and I'm not sure if deploying from branches is supported. And I think it'd be better to use versioned docs since we could modify both the current and the Regarding bugfixes, it's already the case, though we only did it if it's necessary, see
While it's a nice thing to have, disabling OTP (which is required for nightly build) does not look like a good idea.
Our current versioning is not that strict, we could try a stricter one in the next release, that is, patch releases are only used for significant bugfixes (which usually requires |
it is supported... it's actually what happens with every merge request - when you get the preview URL.
true, but what would be the usecase? updating the docs of 1.x and 2.x at the same time will mean also releasing at the same time and having only one release branch... makes it kinda tough. Most bigger projects have branches for each major version so I guess you will need that at some point anyways and having a "next" branch (seems to be currently the most common name for it) would be just one other release branch. You could use it for nightly builds but as said before you can skip that and say install via I would even say that's easier to understand...
Overall I would still say the benefits for multiple release branches are worth the extra effort. PS: not every PR needs to go next -> master; you can go to master directly and then rebase |
Thanks, I just found it on Netlify:
I meant the next minor version, not the major one. Of course the next major version should be in a separate branch. |

Context: #5629 (comment)
Doc preview: https://deploy-preview-5671--prettier.netlify.com/docs/en/cli.html
Diff with a commit that preceded the addition of
--check(to confirm that I did not forget to remove something from the docs folder): f94f63b...kachkaev:temporary-hide-check-optionIt'd be great if the current docs were based on the current version of prettier to avoid confusion. How about using
masterbranch for patches / current docs andcanaryfor the future stuff? Next.js folks do that (https://github.com/zeit/next.js/).