-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Flag config options in code to show "for advanced users only" admonition #18408
Comments
I think we can shorten it to an |
How is it planned to notify the users? I propose following:
|
@HonkingGoose |
@betterPeleg Here's your Markdown: 😉 <!-- prettier-ignore -->
!!! warning
Warning, for advanced use only! Use at your own risk! Edit: I think this is better though: <!-- prettier-ignore -->
!!! warning
For advanced use only! Use at your own risk! The admonition text itself already displays "Warning" as title, so we don't need it again in the body text. 😉
@secustor I'm sorry for confusing you. We plan to notify the users via the docs. I don't know if adding a log
I don't like this idea for "advanced users only warnings" in the Dependency Dashboard. It may work somewhat OK for "deprecated" config options. But we're planning to migrate a users' outdated config with a Renovate PR automatically. Once that feature lands we don't need a Dependency Dashboard notification: |
@HonkingGoose Oh, gotcha! |
I think a docs-only warning message is ok. We don't to want to warn advanced users non-stop e.g. in Dashboard |
What would you like Renovate to be able to do?
Right now, we have to put an admonition in the docs to tell users that a config option is "for advanced users only".
@rarkins had the idea of automating this, like we do already for experimental config options.
If you have any ideas on how this should be implemented, please tell us here.
advancedUse
that we can attach to config options in the codewarning
admonition for us.How it should work:
advancedUse=true
then our docs build process adds the admonition to the docsadvancedUse=false
then nothing happens.Related comment:
Is this a feature you are interested in implementing yourself?
No
The text was updated successfully, but these errors were encountered: