Skip to content

Reduce the number of displayed jobs with cluv status - #160

Open
hvdbm wants to merge 3 commits into
masterfrom
status/max-show-jobs
Open

Reduce the number of displayed jobs with cluv status#160
hvdbm wants to merge 3 commits into
masterfrom
status/max-show-jobs

Conversation

@hvdbm

@hvdbm hvdbm commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The jobs table now only show the last 10 jobs instead of all the cluv jobs. The number of jobs to display can be set with --max-jobs.
  • Now correctly show the last submitted jobs first.

Issues

/

@codecov-commenter

codecov-commenter commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.41%. Comparing base (c40c9c4) to head (61c3e0e).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
cluv/cli/status.py 40.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
+ Coverage   76.19%   76.41%   +0.22%     
==========================================
  Files          19       19              
  Lines        2071     2082      +11     
==========================================
+ Hits         1578     1591      +13     
+ Misses        493      491       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hvdbm
hvdbm marked this pull request as ready for review July 30, 2026 13:44
@hvdbm
hvdbm requested a review from lebrice July 30, 2026 13:44
Comment thread cluv/cli/status.py
Comment on lines +496 to +502
def _build_job_table_legend(max_jobs: int, total_jobs: int) -> Panel:
shown_jobs = min(max_jobs, total_jobs)

return Panel(
f"Showing {shown_jobs} / {total_jobs} cluv jobs",
border_style="dim",
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No need to say "Showing 3 / 3 cluv jobs". Also, why would I want to see only 7, 10, or 20 jobs?
Personally what I expected was for this to show a message saying something like "Only showing 10 / 123 jobs. Use --all-jobs to show all jobs." or something similar, only when there are more jobs that are not shown.

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