fix(Configurations): rename RESONANCE_TEST to FTM_RESONANCE_TEST (411 occurrences)#1192
Open
WakinCode wants to merge 1 commit into
Open
fix(Configurations): rename RESONANCE_TEST to FTM_RESONANCE_TEST (411 occurrences)#1192WakinCode wants to merge 1 commit into
WakinCode wants to merge 1 commit into
Conversation
… occurrences) Global rename was missed in the Configurations repo when FTM_RESONANCE_TEST was introduced in the main Marlin repo. All 411 occurrences were bare RESONANCE_TEST with no existing FTM_RESONANCE_TEST references present. Replaced via: grep -rlZ 'RESONANCE_TEST' . | xargs -0 sed -i 's/\bRESONANCE_TEST\b/FTM_RESONANCE_TEST/g' Tested: zero remaining bare RESONANCE_TEST occurrences confirmed by grep. Affects: config/default and all config/examples across all printer variants.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
Global rename was missed in the Configurations repo when FTM_RESONANCE_TEST
was introduced in the main Marlin repo. All 411 occurrences were bare
RESONANCE_TEST with no existing FTM_RESONANCE_TEST references present.
Replaced via:
grep -rlZ 'RESONANCE_TEST' . | xargs -0 sed -i 's/\bRESONANCE_TEST\b/FTM_RESONANCE_TEST/g'
Tested: zero remaining bare RESONANCE_TEST occurrences confirmed by grep.
Affects: config/default and all config/examples across all printer variants.
Benefits
Enables FTM_RESONANCE_TEST to compile correctly when uncommented in any
example config. Previously the bare RESONANCE_TEST define was silently
ignored by the preprocessor, causing the feature to appear configured
but never actually compile in.
Related Issues
None — discovered while testing FTM_RESONANCE_TEST on Ender 3 V2 with
Creality 4.2.2 board.