Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: add scrolling to repo register, fix --repo and update the cli docs #1133

Merged
merged 3 commits into from
Oct 7, 2023

Conversation

rdimitrov
Copy link
Member

@rdimitrov rdimitrov commented Oct 6, 2023

The following PR adds a bit of polishing to the medic cli -

  • Removed the paging while doing repo register. It now lets you have 20 items per screen but you can scroll dynamically through your full list of repos for a better UX.
  • The --repo flag is now working
  • Went through the flag and command descriptions for each call and fixed things to be consistent across the whole CLI
  • Regenerated the cli docs

Fixes: #861

Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
@JAORMX JAORMX merged commit dd1cd89 into mindersec:main Oct 7, 2023
13 checks passed
@lukehinds
Copy link
Contributor

A lot of this is about to be stripped out

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.

medic: fails to register repositories using --repo
3 participants