linutil icon indicating copy to clipboard operation
linutil copied to clipboard

enhancement: Rate mirrors heads up message

Open jeevithakannan2 opened this issue 1 year ago • 5 comments

Pull Request

Title

Rate mirrors heads up message

Type of Change

  • [x] UI/UX improvement

Description

I've noticed that users often cancel the script while rate-mirrors is running, due to the lack of a visible message indicating that the process is ongoing. This results in an empty mirrorlist making it a hell for linux newbies. Adding a heads-up message informing the user that the process may take a couple of seconds to complete can save the day for a linux rookie.

Checklist

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

jeevithakannan2 avatar Sep 11 '24 17:09 jeevithakannan2

@jeevithakannan2 You're going to have to rebase and rename the "commit linutil" to something else and remove the changes to the binary.

ghost avatar Sep 11 '24 18:09 ghost

@jeevithakannan2 You're going to have to rebase and rename the "commit linutil" to something else and remove the changes to the binary.

Solved the issue

jeevithakannan2 avatar Sep 11 '24 18:09 jeevithakannan2

@jeevithakannan2 You're going to have to rebase and rename the "commit linutil" to something else and remove the changes to the binary.

Solved the issue

I wouldn't say the issue is solved, but you won't be given credit in that commit 493fedf since the author is Chris.

ghost avatar Sep 11 '24 18:09 ghost

This PR looks good overall. :+1:

ghost avatar Sep 11 '24 18:09 ghost

Sorry for the inconvenience. We had a massive restructure of the codebase to improve future development. Because of this can you update your PR to the new structure. Thank you for your assistance and contribution.

ChrisTitusTech avatar Sep 12 '24 18:09 ChrisTitusTech