Skip to content

Conversation

Piskoo
Copy link
Collaborator

@Piskoo Piskoo commented Jun 16, 2025

Fix for #2035

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Copy link
Member

@migmartri migmartri left a comment

Choose a reason for hiding this comment

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

Thanks! Did you try this? Do you have the env environment running?

@Piskoo
Copy link
Collaborator Author

Piskoo commented Jun 16, 2025

Thanks! Did you try this? Do you have the env environment running?

Yes, I have the environment up and running, and I tried it, everything looks good on my end.

//Response before the change
# go run app/cli/main.go wf run ls --project myproject --limit 1
WRN API contacted in insecure mode
ERR validation error:
 - pagination.limit: value must be greater than or equal to 2 and less than or equal to 100 [int32.gte_lte]```
 
//After
 
# go run app/cli/main.go wf run ls --project myproject --limit 1
WRN API contacted in insecure mode
┌──────────────────────────────────────┬────────────────┬─────────────────────┬─────────────┬─────────────────────┬─────────────┐
│ ID                                   │ WORKFLOW       │ VERSION             │ STATE       │ CREATED AT          │ RUNNER      │
├──────────────────────────────────────┼────────────────┼─────────────────────┼─────────────┼─────────────────────┼─────────────┤
│ d87a81af-f487-4ba9-bff6-c3893fdc9cae │ myproject/test │ v1.7.0 (prerelease) │ initialized │ 16 Jun 25 10:05 UTC │ Unspecified │
└──────────────────────────────────────┴────────────────┴─────────────────────┴─────────────┴─────────────────────┴─────────────┘```

@migmartri
Copy link
Member

Thanks! Did you try this? Do you have the env environment running?

Yes, I have the environment up and running, and I tried it, everything looks good on my end.

//Response before the change
# go run app/cli/main.go wf run ls --project myproject --limit 1
WRN API contacted in insecure mode
ERR validation error:
 - pagination.limit: value must be greater than or equal to 2 and less than or equal to 100 [int32.gte_lte]```
 
//After
 
# go run app/cli/main.go wf run ls --project myproject --limit 1
WRN API contacted in insecure mode
┌──────────────────────────────────────┬────────────────┬─────────────────────┬─────────────┬─────────────────────┬─────────────┐
│ ID                                   │ WORKFLOW       │ VERSION             │ STATE       │ CREATED AT          │ RUNNER      │
├──────────────────────────────────────┼────────────────┼─────────────────────┼─────────────┼─────────────────────┼─────────────┤
│ d87a81af-f487-4ba9-bff6-c3893fdc9cae │ myproject/test │ v1.7.0 (prerelease) │ initialized │ 16 Jun 25 10:05 UTC │ Unspecified │
└──────────────────────────────────────┴────────────────┴─────────────────────┴─────────────┴─────────────────────┴─────────────┘```

Awesome @Piskoo!

Thanks a lot!

@jiparis jiparis self-requested a review June 16, 2025 17:11
@migmartri migmartri merged commit f377ba7 into chainloop-dev:main Jun 16, 2025
16 of 17 checks passed
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.

3 participants