Fail if config.h
-migrateable entities aren't in keyboard.json
.
#23449
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Now that the
config.h
ordering has been somewhat sorted (#23448, keyboard before generatedinfo_config.h
before keymap), this will ensure that if there are keyboard-level items inconfig.h
cause failures during build, prompting migration.Additionally, a large chunk of documentation will need to be migrated away from promoting the use of
config.h
before we could consider merging this.This is raised early for visibility purposes and is not expected to be merged any time soon; almost all migrations will need to occur first, otherwise there will be hundreds of build failures.
Types of Changes
Checklist