Skip to content
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

Rename generically named sway functions #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

midirhee12
Copy link

There were some functions that had _hypr and the sway equivalent did not have _sway at the end. This leads the reader of the code to believe that it is generically used for both wayland compositors. So I have renamed all occurrences to respect that all sway related functions shall end with _sway instead.

@midirhee12
Copy link
Author

This PR helps further along #30

@midirhee12 midirhee12 changed the title Renames generically named sway functions Rename generically named sway functions Jul 3, 2024
@nwg-piotr
Copy link
Owner

Convert this PR to draft, please. I can't focus on reviewing every step, as I'm trying to do my own job at the same time.

@midirhee12 midirhee12 marked this pull request as draft July 3, 2024 00:13
@midirhee12
Copy link
Author

Btw, this PR doesn't require any testing for riverwm. It should be functionally the same as current master but with some minor renaming of things.

@nwg-piotr
Copy link
Owner

I am aware of this.

@midirhee12 midirhee12 marked this pull request as ready for review July 27, 2024 19:25
@midirhee12
Copy link
Author

I assume since there's been no changes for the last month that it is probably ready for review. I can change it back to draft if it needs more changes.

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