Azure / Azure/azure-rest-api-specs
[BUG] Get commits $skip parameter is ignored
- Dominant language
- TypeSpec
- Stars
- 3.1k
- Forks
- 5.9k
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 424
Description
### API Spec link
https://learn.microsoft.com/en-us/rest/api/azure/devops/git/commits/get-commits?view=azure-devops-rest-7.1&tabs=HTTP#on-a-branch
### API Spec version
7.1
### Describe the bug
the $skip parameter is ignored
### Expected behavior
to match the documentation
### Actual behavior
```
$ curl -H "authorization: bearer $token" -H "content-type: application/json" -X GET "https://dev.azure.com/Clarios/SW_Euler/_apis/git/repositories/Euler/commits?api-version=7.1&searchCriteria.$top=10"
| jq | head
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 67531 100 67531 0 0 83947 0 --:--:-- --:--:-- --:--:-- 83889
{
"count": 100,
"value": [
{
"commitId": "d926c886eaa6bc5e765840f77c8cbfbd82db1910",
"author": {
},
$ curl -H "authorization: bearer $token" -H "content-type: application/json" -X GET "https://dev.azure.com/Clarios/SW_Euler/_apis/git/repositories/Euler/commits?api-version=7.1&searchCriteria.$top=10&searchCriteria.$skip=1" | jq | head
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 67531 100 67531 0 0 146k 0 --:--:-- --:--:-- --:--:-- 146k
{
"count": 100,
"value": [
{
"commitId": "d926c886eaa6bc5e765840f77c8cbfbd82db1910",
"author": {
},
```
### Reproduction Steps
can be taken from actual behavior snippet
### Environment
_No response_
Contributor guide
Research direction
Start with the linked Azure DevOps Get Commits API specification and search this repository for the commits endpoint definition. Compare requests with and without searchCriteria.$skip, then verify that the returned commits differ as documented and that the specification accurately describes the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- openapi
- Domain
- api
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100