-
Notifications
You must be signed in to change notification settings - Fork 16.2k
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
docs: integrations
reference updates 3
#24796
docs: integrations
reference updates 3
#24796
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
integration
update referencesintegrations
reference updates
239ef74
to
30139c4
Compare
@efriis Erick, could you, please, make the lint error more informative? Thanks!
|
5d45f6c
to
e696328
Compare
integrations
reference updatesintegrations
reference updates 3
60863da
to
6b6965f
Compare
6b6965f
to
f43104e
Compare
@hwchase17 It is tempting to enforce a template to new examples. But it doesn't work if the template is not tested on the old examples. I found no old LLM example that fits this new template. How about "eating dog food" before enforcing? |
Hey @leo-gan ! We're only enforcing this for newly contributed docs. The linked issue in the error message outlines exactly how to bootstrap this document and the template to match Let me know if that doesn't make sense |
@efriis Erick,
|
Appreciate the feedback. Which heading is creating the most issues for you? Updated the error message to be more clear which headings are missing in #25782 |
@efriis "Setup" and "Installation" from the template is usually a single section "Setup and installation" or "Installation and Setup". Both should be OK, because it is clear for the reader. To summarize:
|
got it, and thank you for writing up. Let's stick with the templates for now (and fix this PR if it's desired), as we'd like to standardize the format a bit more than you outlined. |
Righ now the error:
|
00dccd7
to
b65ab0f
Compare
as a heads up, building the docs locally will yield the same broken link errors vercel does |
Added missed provider pages and links. Fixed inconsistent formatting.