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
spec: behavior of required #1024
Comments
using
if you add |
just tested it with ERROR, works for me, but the configuration of spec really is confusing because there are so many options to set |
Can you please bring a full example how you tested it with ERROR? |
And what about the disappear of |
And what is the difference of |
by setting |
This was not explained in METADATA.ini, please update. I would suggest to rename
Does not work for me, I have both (conflict/{get,set} to ERROR (there are range and conflict, subkeys though)), and simplespeclang yields |
|
or per-key config:
|
For me
I wonder how it can fail at all, I thought global plugins do not have a way to fail
So import works even though keys are missing:
|
|
|
which is, in this case, not a good idea,
Yes, I know it works well for small examples. But with multiple keys (where some of that are set) and multiple plugins involved (e.g. enum) there are plenty of issues. It is hard to reproduce something minimalistic but I think if you do whats shown in the tutorial you will find the issues easily. |
ok, found the issue. for my intent was for keys like |
Is it easy to be fixed? |
not really, it's more like an different approach and works fine for what it does (e.g require every array member to have a specific subkey) |
Can we make "postcommit/user/plugins/#0/conflict/get = ERROR" the default please? I would like that And |
In #2705 (comment) Markus wrote:
While it is true that having defaults everywhere is preferrable, LCDproc is actually the perfect example for when A simple example are the Another (even clearer) example, is the ring parameter type. It lets the user select from a set of predefined strings. These strings are given as an array in the config. These strings are per definition |
Ok, if |
I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue. |
I closed this issue now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue. |
Describe what you wanted to do
I wanted to enforce the presence of some configuration values.
Thus I added
required
as meta-data of the relevant configuration entries.For example:
Describe what you expected
required
is appended to the keys.Describe what actually happened
System Information
Further Log Files and Output
The text was updated successfully, but these errors were encountered: