-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Hide fewer-braces page. #15336
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
Hide fewer-braces page. #15336
Conversation
It only hides the page on reference documentation but the PR is targeted to main, so this change won't affect neither the dotty.epfl.ch nor docs.scala-lang.com until release |
How do we hide a page that's no longer relevant? |
It's just matter of the sidebar.yml in which the change is introduced. This one is dedicated for reference documentation. Applying this change to |
Yes, the PR targets main because it reflects the current state (not yet released) of the compiler. I chose to hide the page from the sidebar to not break possible links from the outside. |
Makes sense. |
But shouldn't we hide the page on both sidebars? |
I’d like to ask Martin about this change then, since that would impact dotty.epfl.ch. On my side, I’m fine with removing the page from dotty.epfl.ch because I think this website is more like a “live” documentation of the nightly state of the compiler. |
The page can be hidden from both sidebars. |
OK, I pushed another commit to align both websites. They now both render the page, but hide them from the sidebar. |
See b1a6867.