-
Notifications
You must be signed in to change notification settings - Fork 55
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
Update index.md #30
Comments
@sam-mccall @kadircet WDYT? |
yes, that definitely SGTM! |
which one do you prefer, 1 or 2? |
ah sorry they both felt like the same thing at first glance. I suppose having a page which describes features at a glance, with links to pages with more details whenever possible would be great. (i think that's what you mean by first option) If we can generate such a page automatically (i suppose that's what you mean by the second one?), that would be even better. i am just unaware if there are any solutions offered by github-pages for such a thing though, and i am not sure if it would be worth chasing something more complex. So I'd definitely opt for first one, unless there's an easy solution for the second. |
Is this issue still open can i take up this issue ? |
@Palaksharma23 yes, I am not aware of the progress for it, you can definitely take it up! Feel free to add me as a reviewer for the PRs. |
It may actually make sense for the index page be the same as the features page. |
I don't think it should be exactly the same page as the features simply because feature overview doesn't explain what clangd is at all and goes directly into details. But overall, yes, I think it should be possible to just merge two of them. |
Yeah, merging them sounds like a good idea, will |
Hi, @kirillbobyrev , I did a proof of concept at #83 . Have a look and tell me what do you think! |
The index page looks empty and is not really useful for understanding what Clangd is/how it helps you. Completion is cool but does not advertise Clangd much from the first sight (compiler and Clang-Tidy diags + rename might do that). We can either:
The text was updated successfully, but these errors were encountered: