Skip to content
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

Merged
merged 10 commits into from
Jun 25, 2023
Merged

Update docs for Agent v3.48.0 #2196

merged 10 commits into from
Jun 25, 2023

Conversation

DrJosh9000
Copy link
Contributor

No description provided.

@buildkite-docs-bot
Copy link
Contributor

Preview URL: https://2196--bk-docs-preview.netlify.app

@DrJosh9000 DrJosh9000 force-pushed the jd/agent-release/v3.48.0 branch 2 times, most recently from 8bd8950 to ccdad98 Compare June 8, 2023 03:42
@DrJosh9000 DrJosh9000 requested a review from a team June 8, 2023 04:13
pages/agent/v3.md Outdated Show resolved Hide resolved
Copy link
Contributor

@moskyb moskyb left a 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

pages/agent/v3.md Outdated Show resolved Hide resolved
pages/agent/v3.md Outdated Show resolved Hide resolved
pages/agent/v3.md Outdated Show resolved Hide resolved

## Promoted experiments

We've recently promoted some experiments into full features.
Copy link
Contributor

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?

Copy link
Contributor

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?

Copy link
Contributor Author

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"?

Copy link
Contributor

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
Copy link
Contributor

@mbelton-buildkite mbelton-buildkite left a 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. 😊

pages/agent/v3.md Outdated Show resolved Hide resolved
pages/agent/v3.md Outdated Show resolved Hide resolved
Co-authored-by: Michael Belton <119824349+mbelton-buildkite@users.noreply.github.com>
@DrJosh9000 DrJosh9000 merged commit a8563d1 into main Jun 25, 2023
@DrJosh9000 DrJosh9000 deleted the jd/agent-release/v3.48.0 branch June 25, 2023 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants