Skip to content

Conversation

@DavidGrashton
Copy link
Contributor

Correcting syntax of the notify_by parameter

What does this PR do? What is the motivation?

Updates "notify_by": [] to "notify_by": [""] for accuracy

Merge instructions

Merge readiness:

  • [√] Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

Correcting syntax of the notify_by parameter
@DavidGrashton DavidGrashton requested a review from a team as a code owner May 2, 2025 19:33
@github-actions github-actions bot added the Guide Content impacting a guide label May 2, 2025
@estherk15
Copy link
Contributor

/merge

@dd-devflow
Copy link

dd-devflow bot commented May 2, 2025

View all feedbacks in Devflow UI.

2025-05-02 19:36:48 UTC ℹ️ Start processing command /merge


2025-05-02 19:36:53 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2025-05-02 23:37:05 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@estherk15 estherk15 merged commit 911d73d into master May 15, 2025
9 checks passed
@estherk15 estherk15 deleted the DavidGrashton-patch-2 branch May 15, 2025 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Guide Content impacting a guide mergequeue-status: removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants