-
-
Notifications
You must be signed in to change notification settings - Fork 482
fix: Update lockscreen.sh path in .config/hypr/keybindings.conf
#1391
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
base: dev
Are you sure you want to change the base?
Conversation
|
This PR is targeting According to our Quarterly Release Policy:
Required Action:
If this is an emergency fix, please add a comment explaining why it needs to target This is an automated message enforcing our quarterly release workflow. |
|
It should work though unless users change the $PATH or not using uwsm. also can you use this command instead: |
Oh ok that makes sense! So by that logic, should every other bind using HyDE scripts use |
|
Yes! can you edit the pr and convert them all. But can you also check for any regressions??? Thank you! |
|
Done! I will be checking for any regressions shortly! Should I do so in the hydevm? Or would just manually testing the keybinds on my local machine be fine? |
|
doing it in a vm will be the best option. Just take your time. |
Pull Request
Description
Please read these instructions and remove unnecessary text.
Type of change
Please put an
xin the boxes that apply:Checklist
Please put an
xin the boxes that apply:Screenshots
(if appropriate)
Additional context
Add any other context about the problem here.
In the
.config/hypr/keybindings.conffile, the binding for executing thelockscreen.shscript is missing$srcPath/at the beginning, making this binding not work out of the box.