Skip to content

lua: change permissions stubs ("allow" -> "mode") - #14400

Merged
vaxerski merged 1 commit into
hyprwm:mainfrom
alba4k:patch-6
May 10, 2026
Merged

lua: change permissions stubs ("allow" -> "mode")#14400
vaxerski merged 1 commit into
hyprwm:mainfrom
alba4k:patch-6

Conversation

@alba4k

@alba4k alba4k commented May 10, 2026

Copy link
Copy Markdown
Contributor

Describe your PR, what does it fix/add?

It fixes the lua lsp suggesting stuff like hl.permission({binary = "a", type = "b", allow = "c"}) instead of hl.permission({binary = "a", type = "b", mode = "c"})

Is it ready for merging, or does it need work?

should be good, unless this needs to be changed somewhere else? wiki is fine

furthermore, hl.monitor seems to expect scale as a string according to the lsp (unrelated to this mr), but I haven't found where that is coming from

@github-actions

Copy link
Copy Markdown

Hello and thank you for making a PR to Hyprland!

Please check the PR Guidelines and make sure your PR follows them.
It will make the entire review process faster. :)

If your code can be tested, please always add tests. See more here.

beep boop, I'm just a bot. A real human will review your PR soon.

@vaxerski
vaxerski merged commit b740add into hyprwm:main May 10, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants