Conversation
This fixes the following warnings: warning: line 24: It's not recommended to have unversioned Obsoletes: Obsoletes: python3.11-ovirt-engine-sdk4 warning: line 25: It's not recommended to have unversioned Obsoletes: Obsoletes: python3.12-ovirt-engine-sdk4 Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
As distutils is removed in Python 3.12, replace the functionality with setuptools.Extension. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
Add build for CentOS Stream 10 Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
Drop python3-pycodestyle and python3-pyflakes as those are not used anymore in the project. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
Member
Author
|
Added python3-nose2 to virt10s-ovirt-45-testing and virt9s-ovirt-45-testing |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
dupondje
force-pushed
the
c10s
branch
13 times, most recently
from
72b7eb8 to
df92628
Compare
peter-boden
approved these changes
May 13, 2025
A lot of things needed to be excluded. Used https://github.com/github/gitignore/blob/main/Python.gitignore as a base. Removed the egg-info folder which should not be in git. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
- As python-nose is deprecated, move all tests to pytest. - Add tox to run the tests. - Add flake8 as linter - Run tox during CI. - Let the HTTP server return Content-Length for tests. As otherwise tests are broken since newer OpenSSL version [1] [1]: curl/curl#13389 Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
Fix all the lint errors reported by flake8. Exclude E131 and E501. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
Adjust the setup.py with 'oVirt Developers' as authors and use the new license argument. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
peter-boden
approved these changes
May 13, 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
4 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.
Fixes to support c10s