Skip to content

Conversation

@pquentin
Copy link
Member

@pquentin pquentin commented Oct 9, 2025

We're no longer using the GitHub API in this script.

@pquentin pquentin requested a review from a team as a code owner October 9, 2025 10:37
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2025

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
bulk 🟢 536/536 → 537/537 554/554 → 555/555
index 🟢 1435/1435 → 1437/1437 1437/1437 → 1439/1439
indices.create 🔴 1353/1378 → 1356/1380 1378/1378 → 1380/1380
indices.flush 🟢 52/52 → 54/54 52/52 → 54/54
indices.get_mapping 🔴 197/197 → 198/198 188/197 → 187/198
indices.put_mapping 🟢 145/145 → 147/147 148/148 → 150/150
indices.refresh 🟢 327/327 → 328/328 327/327 → 328/328
search 🔴 2581/2608 → 2584/2611 2608/2608 → 2611/2611

You can validate these APIs yourself by using the make validate target.

@pquentin pquentin merged commit b98fd2d into main Oct 9, 2025
11 of 12 checks passed
@pquentin pquentin deleted the validate-pr-token branch October 9, 2025 11:12
github-actions bot pushed a commit that referenced this pull request Oct 9, 2025
github-actions bot pushed a commit that referenced this pull request Oct 9, 2025
pquentin added a commit that referenced this pull request Oct 9, 2025
(cherry picked from commit b98fd2d)

Co-authored-by: Quentin Pradet <[email protected]>
pquentin added a commit that referenced this pull request Oct 9, 2025
(cherry picked from commit b98fd2d)

Co-authored-by: Quentin Pradet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants