-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
Add revision 1 and revision 2 to ProjectKB Alice PCB #8718
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So rules.mk can go either way, either
keyboards/projectkb/alice/rules.mk
has all the common stuff, feature enabled/disabled logic and the revrules.mk
contains just overrideskeyboards/projectkb/alice/rules.mk
is kinda empty (in this case just the default folder) and the revrules.mk
contains everything
Would be happy with either, maybe a slight preference for 2.
I prefer Option 2 because eventually I hope we can completely do away with |
Co-Authored-By: James Young <[email protected]>
Co-Authored-By: James Young <[email protected]>
@zvecr @noroadsleft thanks again for the feedback! Updated. |
Thanks! |
* upstream/master: (1967 commits) [Docs] Japanese translation of docs/feature_combo.md (qmk#8671) Remove old setup scripts (qmk#8811) [Keyboard] ProjectKB Alice rev2 (qmk#8718) MILC: Fix setting config values for store_true and store_false (qmk#8813) KC60 Refactor (qmk#8782) Fix quick7 build command (qmk#8812) Miscellaneous stuff: remove clang-complete and autocomplete.sh, fix some rules.mk comments (qmk#8784) [Keyboard] add new keyboard quick7 (qmk#8670) [Keymap] Added custom layout for KBDFans KBD67 rev2 (qmk#8752) [keyboard] Add kudox rev3 (qmk#8704) CLI: Fix doctor error when can't run `bin/qmk --version`. (qmk#8796) deleted unused keymap (qmk#8797) Add missing repository name to qmk setup w/ github (qmk#8792) Configure RGBLIGHT_SPLIT for Mint60 (qmk#8788) [Keyboard] Aplx6 (qmk#8727) update pvinis keymap (qmk#8746) CLI: Use `shutil.which` to detect gmake, instead of OS check. CLI: Invoke gmake on FreeBSD when using `qmk compile`. [Docs] Japanese translation of docs/feature_dip_switch.md (qmk#8673) Fix broken KBD8X MKII Hardware link (qmk#8787) ...
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
* Add revision 1 and revision 2 to ProjectKB Alice PCB * Swap SLEEP LED to no * Basic root rules.mk * Apply suggestions from code review * Update keyboards/projectkb/alice/rules.mk
Description
Adding a new revision of the ProjectKB Alice PCB
Types of Changes
Issues Fixed or Closed by This PR
Checklist