Skip to content
This repository has been archived by the owner on Feb 15, 2024. It is now read-only.

Various improvements #270

Merged
merged 8 commits into from
Nov 22, 2023
Merged

Various improvements #270

merged 8 commits into from
Nov 22, 2023

Conversation

JakubFrejlach
Copy link
Collaborator

Originally this PR aimed for the GRIF-118 and a lot of additional improvements were done while working on this, however GRIF-118 was recently proclaimed as obsolete due to a change of process and thus this PR brings only the improvements and not the actual GRIF-118 implementation.

Changes:

  • griffon example plugin
  • <OSIDB/CORGI>_API_URL changed to <OSIDB/CORGI>_SERVER_URL as this was confusing for a lot of users
  • style and color helpers were created to better render text which cannot be rendered using rich library
  • generic GriffonException was created to standardize exception across griffon
  • custom options and arguments were created to simplify mutually exclusive and one of options
  • progress bar was standardized and no_progress_bar was fixed

This caused a lot of confusion whether to assign OSIDB/Corgi API URL
or more generally server URL. Since Griffon only needs general url
name of these variables were changed
This helper is supposed to format text color/style in cases where rich
cannot be used (exceptions for example)
this also fixes non working no progress bar option
Copy link
Collaborator

@JimFuller-RedHat JimFuller-RedHat left a comment

Choose a reason for hiding this comment

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

LGTM

@JakubFrejlach JakubFrejlach merged commit 26324d8 into main Nov 22, 2023
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants