Skip to content

[SDCD-2401] Document additional DORA event fields - #39509

Merged
Tecoddy merged 1 commit into
masterfrom
teddy.gesbert/dora_new_fields
Aug 28, 2026
Merged

[SDCD-2401] Document additional DORA event fields#39509
Tecoddy merged 1 commit into
masterfrom
teddy.gesbert/dora_new_fields

Conversation

@Tecoddy

@Tecoddy Tecoddy commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Fixes SDCD-2401

Documents additional pull request and commit fields collected by DORA Metrics, including their product requirements and GitHub availability.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

AI assistance was used to compare the documented fields with the provided schema and draft concise field descriptions.

Additional notes

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@joepeeples joepeeples left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved with 1 possible grammatical edit, thanks!

| `Number of Lines Added` | Count of lines added in the pull request. Only available for GitHub. |
| `Number of Lines Deleted` | Count of lines deleted in the pull request. Only available for GitHub. |
| `Total Number of Lines Changed` | Total count of lines added and deleted in the pull request. Only available for GitHub. |
| `Time to First Human Review` | Duration until the pull request receives its first review from a human. Only available for GitHub. |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should be past tense, if the human review has already happened when this event gets created:

Suggested change
| `Time to First Human Review` | Duration until the pull request receives its first review from a human. Only available for GitHub. |
| `Time to First Human Review` | Duration until the pull request received its first review from a human. Only available for GitHub. |

@Tecoddy
Tecoddy marked this pull request as ready for review August 28, 2026 16:32
@Tecoddy
Tecoddy requested a review from a team as a code owner August 28, 2026 16:32
@Tecoddy
Tecoddy merged commit 88c00f8 into master Aug 28, 2026
24 of 25 checks passed
@Tecoddy
Tecoddy deleted the teddy.gesbert/dora_new_fields branch August 28, 2026 16:32
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.

2 participants