feat(aliases): add history, history undo, repolist, repoquery, and ch… - #2408
Waqar-Ahmad76 wants to merge 1 commit into
Conversation
…eck aliases to dnf
seefood
left a comment
There was a problem hiding this comment.
-
Inconsistent indentation / alignment: The # comments use a tab + one space before the text, but the # comment column is misaligned. For example, alias dnfmc="dnf makecache" # Generate metadata cache
uses multiple spaces to pad to a comment column, while the new lines use a tab (or irregular spacing). This breaks the visual alignment pattern established in the existing file. -
clean_files.txt not updated: The checklist explicitly says "If I have added a new file, I also added it to clean_files.txt and formatted it" — they didn't tick that box, but since this is an existing file
(not new), this clause may not strictly apply. Worth checking whether dnf.aliases.bash is already covered by clean_files.txt. -
dnf history undo without -y: The dnf history undo alias will prompt for confirmation each time, unlike dnfi/dnfr which use sudo and are one-offs. Not necessarily a bug — it's reasonable to require
confirmation for undo — but worth calling out.
…eck aliases to dnf
Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
clean_files.txtand formatted it usinglint_clean_files.sh.