-
Notifications
You must be signed in to change notification settings - Fork 300
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
Separate keymaps.h file (rebased) #84
base: master
Are you sure you want to change the base?
Conversation
92b9646
to
193f6a4
Compare
23dd18d
to
fbb414b
Compare
Signed-off-by: Andrew Gallagher <andrew.gallagher@siren.io>
Signed-off-by: Andrew Gallagher <andrew.gallagher@siren.io>
fbb414b
to
afb2172
Compare
Any chance of getting an opinion one way or another on this PR? Keeping it open and mergeable while master constantly changes underneath it is time-consuming. CF original discussion here: #28 |
I'm really sorry for letting this drag out. On the one hand, it's not quite
what I want for the regular Model 01 sketch. On the other hand, it's a
really valuable thing for some folks. Would you be up for resubmitting it
as a new PR against Kaleidoscope adding it as another Model 01 example
sketch?
…On Sat, Mar 7, 2020 at 6:51 AM Andrew Gallagher ***@***.***> wrote:
Any chance of getting an opinion one way or another on this PR? Keeping it
open and mergeable while master constantly changes underneath it is
time-consuming.
CF original discussion here: #28
<#28>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#84?email_source=notifications&email_token=AAALC2HONOUBMBI4GL4W42LRGJNOBA5CNFSM4HCCJEJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOD3KBY#issuecomment-596096263>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALC2ALRG7LSC4QKSWNAU3RGJNOBANCNFSM4HCCJEJA>
.
|
I can do that if you think it will be useful, but from my PoV its main usefulness lies in being able to maintain local modifications to this repo while tracking upstream. Large, single config files mean more merge conflicts. If you think there is a better way of doing this, say by using a config.d directory, I'm all for it. |
This has been hanging out in my inbox as I've been continuing to ponder it.
(Of course, in the meantime, the Model 01 firmware hasn't been seeing many
changes ;)
Being able to override keymaps inside a config directory is certainly a
reasonable pattern and would work well with my desire to have the default
firmware sketch as a single, relatively readable file
It -is- my intent to migrate the default sketch into Kaleidoscope's repo as
we work on reducing the number of places folks need to look to find things,
but it hasn't exactly been urgent.
…On Thu, Apr 16, 2020 at 7:22 AM Andrew Gallagher ***@***.***> wrote:
I can do that if you think it will be useful, but from my PoV its main
usefulness lies in being able to maintain local modifications to this repo
while tracking upstream. Large, single config files mean more merge
conflicts.
If you think there is a better way of doing this, say by using a config.d
directory, I'm all for it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#84 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALC2BFFHPVXBADW3B7C2DRM4IDRANCNFSM4HCCJEJA>
.
|
Same as previous PR #64, but rewritten on top of latest upstream/master.