Skip to content

Add known limitations for inline hint injections#23162

Merged
jhlodin merged 2 commits intomainfrom
jl/doc-16090
Apr 8, 2026
Merged

Add known limitations for inline hint injections#23162
jhlodin merged 2 commits intomainfrom
jl/doc-16090

Conversation

@jhlodin
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin commented Apr 6, 2026

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 6, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit c54cf21
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/69d55429f753c20008e413d8

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 6, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit c54cf21
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/69d55429391b11000874cff2

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 6, 2026

Files changed:

@jhlodin jhlodin requested a review from michae2 April 6, 2026 20:19
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 6, 2026

Netlify Preview

Name Link
🔨 Latest commit c54cf21
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/69d55429ad356b000873bd49
😎 Deploy Preview https://deploy-preview-23162--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

@michae2 michae2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, LGTM!

(There's a similar known limitation around views, cockroachdb/cockroach#166782, which would also be good to document.)

@jhlodin
Copy link
Copy Markdown
Contributor Author

jhlodin commented Apr 7, 2026

Nice, LGTM!

(There's a similar known limitation around views, cockroachdb/cockroach#166782, which would also be good to document.)

Added as well, using the same language

@jhlodin jhlodin changed the title Add known limitation for routine hint injections Add known limitation for inline hint injections Apr 7, 2026
@jhlodin jhlodin changed the title Add known limitation for inline hint injections Add known limitations for inline hint injections Apr 7, 2026
Copy link
Copy Markdown
Contributor

@bsanchez-the-roach bsanchez-the-roach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@michae2 michae2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@jhlodin jhlodin merged commit 395351c into main Apr 8, 2026
6 checks passed
@jhlodin jhlodin deleted the jl/doc-16090 branch April 8, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants