Skip to content

Conversation

Angxddeep
Copy link
Contributor

Type of Change

  • Bug fix

Description

Dnf upgrade is more than enough, dnf update is just a subset of upgrade, so there is really no need for it.

Testing

Works

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@adamperkowski adamperkowski changed the title Bugfix System-update.sh using DNF fix(system-update): remove redundant DNF instructions Oct 29, 2024
Copy link
Collaborator

@adamperkowski adamperkowski left a comment

Choose a reason for hiding this comment

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

Thanks

@Angxddeep
Copy link
Contributor Author

I always get confused between refactor and bugfix

@adamperkowski adamperkowski changed the title fix(system-update): remove redundant DNF instructions refact(system-update): remove redundant DNF instructions Oct 29, 2024
@adamperkowski
Copy link
Collaborator

adamperkowski commented Oct 29, 2024

I always get confused between refactor and bugfix

bugfix fixes an issue that breaks functionality
refactor changes code in a way that COULD fix a POTENTIAL issue / benefits the codebase in some way (readability, performance, etc)

@jeevithakannan2
Copy link
Contributor

Duplicate of #843

@adamperkowski adamperkowski added duplicate This issue or pull request already exists and removed refactor labels Oct 30, 2024
@adamperkowski
Copy link
Collaborator

@Angxddeep This does seem like a duplicate. I'll close this PR and commit your additional changes to #843

adamperkowski pushed a commit to jeevithakannan2/linutil that referenced this pull request Oct 30, 2024
@Angxddeep Angxddeep deleted the Testing-#2 branch November 1, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants