-
Notifications
You must be signed in to change notification settings - Fork 250
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
feat(port): Keyboard format updates, tweaks #4635
Conversation
Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com> Co-Authored-By: Chaosvolt <chaosvolt@users.noreply.github.com> Co-Authored-By: Jianxiang Wang (王健翔) <qrox@sina.com> Co-Authored-By: Rob Kuijper <robkuijper@live.nl>
Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com> Co-Authored-By: Chaosvolt <chaosvolt@users.noreply.github.com> Co-Authored-By: Jianxiang Wang (王健翔) <qrox@sina.com> Co-Authored-By: Rob Kuijper <robkuijper@live.nl>
Autofix has formatted code style violation in this PR. I edit commits locally (e.g: git, github desktop) and want to keep autofix
I do not want the automated commit
If you don't do this, your following commits will be based on the old commit, and cause MERGE CONFLICT. |
…clysmbnteam/Cataclysm-BN into keyboardformat_port_update2
…clysmbnteam/Cataclysm-BN into keyboardformat_port_update2
This reverts commit c2d5431.
…clysmbnteam/Cataclysm-BN into keyboardformat_port_update2
Okay so evidently #4636 contained the file changes for this PR and the other one so these can be closed? According to Kenan at least. I'm going to hope this is the right descision, if anyone spots anything vital in this re-open and fix merge conflicts then, bleh. |
Per request but marking it as WIP afterward. |
Co-Authored-By: KheirFerrum <102964889+KheirFerrum@users.noreply.github.com> Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com> Co-Authored-By: Chaosvolt <chaosvolt@users.noreply.github.com> Co-Authored-By: Jason Fields <jasonfields4@gmail.com>
…clysmbnteam/Cataclysm-BN into keyboardformat_port_update2
Closing this because pull request of @KheirFerrum includes fixed version of this work |
Ports these pull requests
CleverRaven/Cataclysm-DDA#40961,
CleverRaven/Cataclysm-DDA#41866,
CleverRaven/Cataclysm-DDA#47263,
CleverRaven/Cataclysm-DDA#56212,
CleverRaven/Cataclysm-DDA#46796,
CleverRaven/Cataclysm-DDA#55152,
CleverRaven/Cataclysm-DDA#57266,
This also includes pull requests #4627, #4623 by @chaosvolt for easier mergeability. This depends on #4634 so it cannot be merged until it's merged
activity_handlers.cpp
file, line 1257,crafting_gui.cpp
file, line 718 throw errors during compiling which, based on conversation with @Coolthulhu, will be fixed with help of Coolthulhu or other developers or both.action.cpp
file, line 930 could not be ported because it throws an error so it was added to a comment with explanationCoauthors have been linked properly via GitHub Desktop built-in function to do this work hopefully