Skip to content

Python3 code fixes and Documentation#5

Open
mshedsilegx wants to merge 3 commits intosergiomb2:mainfrom
mshedsilegx:main
Open

Python3 code fixes and Documentation#5
mshedsilegx wants to merge 3 commits intosergiomb2:mainfrom
mshedsilegx:main

Conversation

@mshedsilegx
Copy link

1- Fixed non working portion of the code due to differences between kid and genshi templating engines (notably rss feeds)
2- Resolved all linting issues generated by pyright, pylint and ruff
3- Added documentation to understand code structure and usage
4- Tested resulting code on a production repo we manage for RHEL 8 / 9 / 10 (about 6000 packages, python 3.13 used with repoview)

@sergiomb2
Copy link
Owner

sergiomb2 commented Dec 30, 2025

Packaging Configuration

and Documentation
merged

about the other Fix python3 code issues, and resolved all linting errors detected by pyright, pylint and ruff I prefer the version in 3 commits, I merged two , I will verify "linter fixes" [1] tommorow

[1]
e98dc24

@mshedsilegx
Copy link
Author

I would be interested in your advice and assessment of the PR quality, so I can adjust/improve things on my end. Appreciate your responsiveness and consideration !

@sergiomb2
Copy link
Owner

sergiomb2 commented Dec 30, 2025

to help please rebase your main repo on top of sergiomb2:main and than break last commit into multiple commits.

git rebase --autosquash -i HEAD~8
replace pick with edit (e for short)

git reset HEAD~
git rebase --continue

reference https://stackoverflow.com/questions/6217156/break-a-previous-commit-into-multiple-commits

@sergiomb2 sergiomb2 force-pushed the main branch 2 times, most recently from b080446 to 9017915 Compare December 31, 2025 03:57
@mshedsilegx
Copy link
Author

I understand now what you wanted. I'll make sure in the future to do smaller commits, each being a specific functionality.

@sergiomb2
Copy link
Owner

I had redo all latest commits , I did also branch pull_6 checkout it out , I´d like split all format string into one commit for example

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.

2 participants