Skip to content

[collector] resolve regex library warnings#4021

Closed
emmanuel-ferdman wants to merge 1 commit into
sosreport:mainfrom
emmanuel-ferdman:main
Closed

[collector] resolve regex library warnings#4021
emmanuel-ferdman wants to merge 1 commit into
sosreport:mainfrom
emmanuel-ferdman:main

Conversation

@emmanuel-ferdman

Copy link
Copy Markdown
Contributor

Please place an 'X' inside each '[]' to confirm you adhere to our Contributor Guidelines

  • Is the commit message split over multiple lines and hard-wrapped at 72 characters?
  • Is the subject and message clear and concise?
  • [ x Does the subject start with [plugin_name] if submitting a plugin patch or a [section_name] if part of the core sosreport code?
  • Does the commit contain a Signed-off-by: First Lastname email@example.com?
  • Are any related Issues or existing PRs properly referenced via a Closes (Issue) or Resolved (PR) line?
  • Are all passwords or private data gathered by this PR obfuscated?

PR Summary

This small PR resolves the regex library warnings showing in Python3.13:

'count' is passed as positional argument

@packit-as-a-service

Copy link
Copy Markdown

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/sosreport-sos-4021
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@TurboTurtle TurboTurtle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code change LGTM, but please squash your commits and put the PR message in the commit message, as part of our Contribution Guidelines

@TurboTurtle TurboTurtle added Kind/Enhancement Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer labels May 21, 2025
This resolves regex library warnings showing in Python3.13 where
'count' is passed as positional argument.

Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com>
@emmanuel-ferdman

Copy link
Copy Markdown
Contributor Author

@TurboTurtle no problem, done 😄

@pmoravec

Copy link
Copy Markdown
Contributor

Fixed via 60afa0e .

@pmoravec pmoravec closed this May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind/Enhancement Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants