-
-
Notifications
You must be signed in to change notification settings - Fork 42.4k
Fix number of elements in info.json does not match errors #14213
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
Conversation
|
If you are tagged in this comment you have a keyboard changed in this PR: @evyd13 @joedinkle @stevendlander @jackytrabbit @melgeek001365 @yiancar @mm0nte @2Moons-JP @projectcain @sleepdealr @vanillakeyboards |
|
@skullydazed the problem with the atom47 seems to be in the matrix definition, not the info.json. Should I open a PR to fix that instead? |
Whatever is easiest for you works for me, as long as we clear up the lint errors. I can remove these changes in favor of a separate PR from you, or you can PR against this branch. |
b2f58ba to
d4b1043
Compare
d4b1043 to
d701e9e
Compare
* evyd13/atom47/rev3: best guess at fixing info.json * evyd13/atom47: best guess at fixing info.json * eyeohdesigns/sprh: best guess at fixing info.json * handwired/unicomp_mini_m: best guess at fixing info.json * lazydesigners/the60/rev2: best guess at fixing info.json * melgeek/mj6xy: best guess at fixing info.json * pizzakeyboards/pizza65: best guess at fixing info.json * plx: best guess at fixing info.json * rojectcain/vault45: best guess at fixing info.json * sidderskb/majbritt/rev2: best guess at fixing info.json * spacey: best guess at fixing info.json
Description
Makes the number of elements in info.json match the number of elements in the C layout.
Types of Changes
Checklist