[Schema Inaccuracy] Webhook auto merge for rebase commits
@nickfloyd ci sta già lavorando.
Dal 6/11/2025.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Schema Inaccuracy
auto_merge.commit_title and auto_merge.commit_message are nullable in practice, but not in the schema.
This is a real webhook from GitHub.com on 2023-04-21.
"auto_merge": {
"enabled_by": {
"login": "zegl",
"id": 47952,
"node_id": "MDQ6VXNlcjQ3OTUy",
"avatar_url": "https://avatars.githubusercontent.com/u/47952?v=4",
"gravatar_id": "",
"url": "https://api.github.com/users/zegl",
"html_url": "https://github.com/zegl",
"followers_url": "https://api.github.com/users/zegl/followers",
"following_url": "https://api.github.com/users/zegl/following{/other_user}",
"gists_url": "https://api.github.com/users/zegl/gists{/gist_id}",
"starred_url": "https://api.github.com/users/zegl/starred{/owner}{/repo}",
"subscriptions_url": "https://api.github.com/users/zegl/subscriptions",
"organizations_url": "https://api.github.com/users/zegl/orgs",
"repos_url": "https://api.github.com/users/zegl/repos",
"events_url": "https://api.github.com/users/zegl/events{/privacy}",
"received_events_url": "https://api.github.com/users/zegl/received_events",
"type": "User",
"site_admin": false
},
"merge_method": "rebase",
"commit_title": null,
"commit_message": null
},
The schema looks like this:
"auto-merge": {
"title": "Auto merge",
"description": "The status of auto merging a pull request.",
"type": "object",
"properties": {
"enabled_by": {
"$ref": "#/components/schemas/simple-user"
},
"merge_method": {
"type": "string",
"description": "The merge method to use.",
"enum": [
"merge",
"squash",
"rebase"
]
},
"commit_title": {
"type": "string",
"description": "Title for the merge commit message."
},
"commit_message": {
"type": "string",
"description": "Commit message for the merge commit."
}
},
"required": [
"enabled_by",
"merge_method",
"commit_title",
"commit_message"
],
"nullable": true
},
This likely also applies to non webhook payloads.
Expected
commit_title and commit_message should be nullable.
Reproduction Steps
Trigger auto-merge with the rebase merge method, and watch the web hooks.
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 1.6k
- Fork
- 342
- Merge medio
- 3h 33m
- PR unite (30g)
- 51
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di github/rest-api-description
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
github/rest-api-description#7201 ·
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
github/rest-api-description#7163 ·
-
Runner deprecations: registration_deprecates_at is declared on the response but never returned Apertafeature
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
github/rest-api-description#7162 ·
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
github/rest-api-description#7135 ·
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
github/rest-api-description#7111 · 1 commento ·