-
Notifications
You must be signed in to change notification settings - Fork 7.8k
build: freshness #23172
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
base: main
Are you sure you want to change the base?
build: freshness #23172
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't finish reviewing yet, but I noticed a few places where the text seems to take shortcuts that subtly change the meaning of core concepts or lose important nuance. For pages like concepts, which are relied on for precision (especially by folks deep in the ecosystem) it's really important to preserve the intent and technical accuracy of the original terms.
Also, looking at the freshness prompt files, I'm guessing this might have been generated with the help of an AI assistant? If so, it'd be helpful to split AI-generated changes and your own edits into separate commits, so it's easier to see where the AI may have gone off track.
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
Co-authored-by: CrazyMax <[email protected]>
It's indeed mostly AI-touched up to see how it runs our styleguide on the content. Maybe that's a bit of waste of time (and review time) for the outcome. I'm not super convinced with this initiative. |
Yes me too but that's good to test it out and maybe improve the model and prompts if we can. |
@crazy-max, any chance you could take another look this week? 🙇 |
Apply various linguistic efficiency improvements.