Allow hiding deprecated config options#2853
Open
peterebden wants to merge 1 commit intothought-machine:masterfrom
Open
Allow hiding deprecated config options#2853peterebden wants to merge 1 commit intothought-machine:masterfrom
peterebden wants to merge 1 commit intothought-machine:masterfrom
Conversation
|
This issue has been automatically marked as stale because it has not had any recent activity in the past 90 days. It will be closed if no further activity occurs. If you require additional support, please reply to this message. Thank you for your contributions. |
peterebden
added
the
not-stale
Collaborator
Author
|
Hush, bot. I'm not sure what the immediate future is here - the main issue is that I'm not sure how to update the C++ plugin to work in the new world as it did before. We need to figure that out anyway though. |
peterebden
force-pushed
the
deprecated-config-options
branch
from
b262a5e to
155a7a3
Compare
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.
This marks all the language-specific stuff as deprecated and adds a feature flag that stops asp from getting them on its config object.
Hopefully this will start cutting down on errors from these (e.g. #2851 ran across another one that we never updated and which had been silently using the wrong Go tool for months); we should find out immediately that these don't exist.