Skip to content

Update Time Travelling Queries Guide #161

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ashutosh-src
Copy link
Contributor

@ashutosh-src ashutosh-src requested review from jsimnz and addobot April 5, 2025 07:02
@ashutosh-src ashutosh-src self-assigned this Apr 5, 2025
Copy link

Deploying docs-source-network with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3241d6d
Status: ✅  Deploy successful!
Preview URL: https://61c8fc3b.docs-source-network.pages.dev
Branch Preview URL: https://ashutosh-guides-time-travell.docs-source-network.pages.dev

View logs

Copy link
Member

@jsimnz jsimnz left a comment

Choose a reason for hiding this comment

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

Overall lgtm (more opinions on the LLM use though)

This PR further emphasizes some issue I mentioned in the previous PR about using LLMs for this kind of refactor

  • The overall structure of the document isn't consistent with the rest of the docs. Lots of --- page breaks, and most notably is the inclusion of the FAQ section. Although it has useful information, its entirely disconnected from the rest of the pages formatting.
  • Links, theres lots of sections/references that could be linked to another page/guide/concept doc, but they have been ommited or entirely deleted in some cases, because the LLM doesn't know the context you are using this for.

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