-
Notifications
You must be signed in to change notification settings - Fork 249
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 docs for Agent v3.48.0 #2196
Conversation
94018ee
to
adf9e07
Compare
Preview URL: https://2196--bk-docs-preview.netlify.app |
8bd8950
to
ccdad98
Compare
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.
tentative accept from me, but probably best to wait for @mbelton-buildkite to weight in
Co-authored-by: Ben Moskovitz <ben.m@buildkite.com>
c4615c9
to
59c1cfd
Compare
pages/agent/v3.md
Outdated
|
||
## Promoted experiments | ||
|
||
We've recently promoted some experiments into full features. |
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.
This triggered a couple of things for me:
- Time words like "recently" are tricky in docs so it's good to avoid them.
- This feels like changelog information rather than docs.
BUT, it seems like useful information. Is there anywhere else it could go? Or perhaps just a different heading that doesn't tie it to a time?
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.
Actually, mentioning the relevant versions like you have, helps. Have we gone through this lifecycle with other experiments? If so, how did they get handled?
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.
Good question. I thought this might be the first time we've actually "completed" experiments. But it turns out that there is at least one completed experiment present in this file - output-redactor
(Redacted variables) was de-experimented in mid 2021 (see buildkite/agent#1452).
Past experiments feels like important documentation to have ("why is my agent printing 'The ansi-timestamps experiment has been promoted...'?") and this seemed like the right page to put it. Perhaps it should be a subsection of "Experimental features"?
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.
I've left a suggestion for handling this in #2240. Let me know what you think there.
Co-authored-by: Michael Belton <119824349+mbelton-buildkite@users.noreply.github.com>
* Proposal for promoted agent experiments * Remove emoji update from change
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.
I've left comments about a couple of minor nitpicks I'd missed, but looks good in general. 😊
Co-authored-by: Michael Belton <119824349+mbelton-buildkite@users.noreply.github.com>
No description provided.