buildtest icon indicating copy to clipboard operation
buildtest copied to clipboard

[FEATURE]: add pagination for 'buildtest inspect' command

Open shahzebsiddiqui opened this issue 2 years ago • 0 comments

Please describe your feature request

Let's add pagination support for buildtest inspect command that will be done via option --pager and short option -p to be consistent.

The pagination will be done via rich see https://rich.readthedocs.io/en/stable/console.html#paging.

See related implementation in buildtest report or buildtest buildspec for pagination in files: https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/report.py and https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/buildspec.py

Action Item

  • [ ] Add --pager and -p option in https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/init.py
  • [ ] Add support of pagination for commands buildtest inspect buildspec, buildtest inspect name, buildtest inspect query and buildtest inspect list in https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/inspect.py
  • [ ] Add option to bash completion script https://github.com/buildtesters/buildtest/blob/devel/bash_completion.sh
  • [ ] Add regression test for pagination in https://github.com/buildtesters/buildtest/blob/devel/tests/cli/test_inspect.py
  • [ ] Add a row entry in buildtest help inspect pertaining to pagination in https://github.com/buildtesters/buildtest/blob/devel/buildtest/cli/help.py

Post question in Slack

  • [X] I agree that I posted my question in slack before creating this issue

Is there an existing issue

  • [X] I confirm there is no existing issue for this issue

shahzebsiddiqui avatar Aug 31 '22 20:08 shahzebsiddiqui