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

Implement a basic keybinding collision checker. (ToDo list #337) #348

Merged
merged 1 commit into from
Oct 11, 2019

Conversation

annagrram
Copy link
Collaborator

No description provided.

@annagrram annagrram force-pushed the basic-keybind-collision-checker branch from 5e1fc19 to c1fcc73 Compare October 9, 2019 22:59
@jarun
Copy link
Owner

jarun commented Oct 10, 2019

Wow! This was quick! Thanks for the PR. I will complete this review shortly.

Note for @jarun: Please document this in the Wiki Advanced...

misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
misc/keybind-collision-checker/collision.c Outdated Show resolved Hide resolved
@annagrram annagrram force-pushed the basic-keybind-collision-checker branch 3 times, most recently from 7cbcbc3 to 9b32da1 Compare October 10, 2019 10:41
@annagrram annagrram force-pushed the basic-keybind-collision-checker branch 2 times, most recently from 9ab7ca4 to 2fe8960 Compare October 10, 2019 19:35
@annagrram annagrram force-pushed the basic-keybind-collision-checker branch from 2fe8960 to 27e74e1 Compare October 10, 2019 19:39
@jarun jarun merged commit ab8e3e4 into jarun:master Oct 11, 2019
@jarun
Copy link
Owner

jarun commented Oct 11, 2019

Thank you for your contribution!

@jarun
Copy link
Owner

jarun commented Oct 12, 2019

@annagrram could you make some time to add the the patch for coreutils v8.31 to advcmpv?

@annagrram
Copy link
Collaborator Author

Sure.
I'll take a look later today.

@lock lock bot locked and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants