Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH: List logged dnf transaction cmds as txt or json #2149

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

westurner
Copy link

  • WIP
    • Would this easily forward-port to dnf5?
  • Adds dnf history store --all and dnf history store --all --comments
    • TODO: example of the output

@pep8speaks
Copy link

Hello @westurner! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 73:121: E501 line too long (124 > 120 characters)
Line 392:9: E265 block comment should start with '# '
Line 409:25: E265 block comment should start with '# '
Line 411:74: E262 inline comment should start with '# '
Line 412:29: E265 block comment should start with '# '
Line 415:29: E265 block comment should start with '# '
Line 427:47: E226 missing whitespace around arithmetic operator
Line 430:5: E303 too many blank lines (2)
Line 435:17: E125 continuation line with same indent as next logical line
Line 436:21: E117 over-indented

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