Merge changes from sandrobonazzola/did fork - #390
Merged
Conversation
sandrobonazzola
force-pushed
the
main
branch
6 times, most recently
from
March 28, 2025 11:04
3227fc6 to
e5ec437
Compare
sandrobonazzola
force-pushed
the
main
branch
2 times, most recently
from
April 15, 2025 13:47
ef3ed49 to
62d6b51
Compare
Adding markdown format output to koji plugin. In order to provide a link to the build, `weburl` option is required, pointing to the koji web interface.
It's now possible to use token authentication for connecting to Confluence. Page list is now also available in markdown format. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Recently Red Hat bugzilla SSL certificate is failing verification. In order to allow querying bugzilla while this issue is getting fixed, allowing to skip SSL certificate validation. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixing pylint warnings: W1201: Use lazy % formatting in logging functions (logging-not-lazy) W1202: Use lazy % or % formatting in logging functions (logging-format-interpolation) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixed: W0707: Consider explicitly re-raising using ... (raise-missing-from) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixing: R0205: Class inherits from object, can be safely removed from bases in python3 (useless-object-inheritance) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixing W0622: Redefining built-in (redefined-builtin) While doing it, removing the unused function utils.ascii which was overriding the ascii built-in as the only place where it was used was its test unit. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: W0107: Unnecessary pass statement (unnecessary-pass) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: W1406: The u prefix for strings is no longer necessary in Python >=3.0 (redundant-u-string-prefix) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: W0611: Unused pformat imported from pprint as pretty (unused-import) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: W0104: Statement seems to have no effect (pointless-statement) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: DeprecationWarning: 'maxsplit' is passed as positional argument Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixing: W3101: Missing timeout argument for method 'requests.get' can cause your program to hang indefinitely (missing-timeout) by adding an optional timeout config to the plugins using that method, defaulting to 60 seconds if not set. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Use proper os.path.join dealing with paths and use f-string to make more understandable the desired strings. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: R1724: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it (no-else-continue) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: R1704: Redefining argument with the local name ... (redefined-argument-from-local) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Fixes: R1734: Consider using [] instead of list() (use-list-literal) Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
- Added timeout support - Added rate limit handling - Reduced differences with jira plugin as preparation to refactoring Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Added hyperkitty and public_inbox plugins to the docs. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
The query for github requesting to exclude an organization was badly encoded. Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
Owner
|
@sandrobonazzola, thanks a lot for all these contributions! This is an incredible amount of work! Very nice improvements! I've merged a couple open pull requests before this large one:
Wanted mainly to prevent introducing an extra license because of the |
Collaborator
Author
|
If it makes things easier feel free to force push to my fork. If you prefer to stay on #395 feel free to close this one. |
psss
approved these changes
May 19, 2025
Owner
|
Ok, thanks! Pushed and merged. |
This was referenced May 19, 2025
Closed
This was referenced May 20, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Notable changes: