Skip to content

Make certain optional features actually disablable - #425

Draft
scj643 wants to merge 1 commit into
Keychron:2025q3from
scj643:fix-opt-features
Draft

Make certain optional features actually disablable#425
scj643 wants to merge 1 commit into
Keychron:2025q3from
scj643:fix-opt-features

Conversation

@scj643

@scj643 scj643 commented Dec 8, 2025

Copy link
Copy Markdown

PR to make it easier to disable unwanted features.

Disabling FACTORY_TEST and RETAIL_DEMO_ENABLE can save a bit over 1kb of space.

Also moved FACTORY_TEST_ENABLE and RETAIL_DEMO_ENABLE to their own
lines.
@scj643

scj643 commented Dec 8, 2025

Copy link
Copy Markdown
Author

Actually it's closer to 3k with both disabled and LTO enabled.

@lokher
lokher force-pushed the 2025q3 branch 4 times, most recently from a5e4148 to 7c2357c Compare April 14, 2026 10:21

@EKl40 EKl40 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see nothing wrong with this. It shouldn’t be marked as a draft, since none of the force-pushes mentioned above introduced any conflicts.

@scj643

scj643 commented Jul 20, 2026

Copy link
Copy Markdown
Author

I'll have to look at it because I think there was some things I needed to validate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants