Skip to content

Commit

Permalink
Revert "fix: Fixup waybar's integrated system updater module"
Browse files Browse the repository at this point in the history
  • Loading branch information
JaKooLit authored Jul 8, 2024
1 parent 3e3273e commit 3a2c20d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions config/waybar/modules
Original file line number Diff line number Diff line change
Expand Up @@ -707,7 +707,7 @@
"exec": "checkupdates | wc -l",
"exec-if": "[[ $(checkupdates | wc -l) ]]",
"interval": 15,
"on-click": "if command -v paru &> /dev/null; then kitty -T update paru -Syu; else kitty -T update yay -Syu; fi && notify-send 'The system has been updated'",
"on-click": "kitty -T update paru -Syu || yay -Syu && notify-send 'The system has been updated'",
"tooltip": true,
"tooltip-format": "Left Click: Update System\nArch Linux Only",
},
Expand Down Expand Up @@ -863,4 +863,4 @@
"interval": 86400, // once every day
"tooltip": true,
},
}
}

7 comments on commit 3a2c20d

@ElectroPerf
Copy link
Contributor

Choose a reason for hiding this comment

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

lol what happened

@JaKooLit
Copy link
Owner Author

Choose a reason for hiding this comment

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

Oh crap..... made a mistake...

can push again a PR so you will get the credit?

thanks

@ElectroPerf
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh crap..... made a mistake...

can push again a PR so you will get the credit?

thanks

Lol will do, what about the other two, why dropped them tho

@JaKooLit
Copy link
Owner Author

Choose a reason for hiding this comment

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

Oh crap..... made a mistake...
can push again a PR so you will get the credit?
thanks

Lol will do, what about the other two, why dropped them tho

tried and 3 of distros Ive tried (Arch, Fedora, Gentoo), it has 0.5 to 1 sec delay in launching....

@ElectroPerf
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh crap..... made a mistake...
can push again a PR so you will get the credit?
thanks

Lol will do, what about the other two, why dropped them tho

tried and 3 of distros Ive tried (Arch, Fedora, Gentoo), it has 0.5 to 1 sec delay in launching....

Well thats an easy fix

@ElectroPerf
Copy link
Contributor

Choose a reason for hiding this comment

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

Reland the commit, I'll fix it up

@JaKooLit
Copy link
Owner Author

Choose a reason for hiding this comment

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

Reland the commit, I'll fix it up

I have updated and put back your PR

Please sign in to comment.