-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add support for role qualifications #15
Comments
Hi Frederik, I will change the code according to your last review and then I will start to work on this. Do you think that the way I proposed the change for the human readable report make sense? Because I will code then accordingly. I think we should add this option before the first release. |
I'll give my thoughts one by one.
Yes, that was my thinking.
That would probably be the easiest way to implement this and it would also make sense, I think.
Either that or we could add superscripts, which may be a little more concise? |
Hi, I agree with your third point. It is a good idea to add superscripts. |
I would not include this in the first release:
If you want to go for it anyway, the default value should be "unspecified" and not "equal" in order to avoid unnecessary conflicts. |
Hi Julien, thanks for your comments. I agree with what you are suggesting. It is pretty much what we landed on in another private e-mail discussion. |
I'll summarize our discussion below.
There are some open questions for the human readable report:
The text was updated successfully, but these errors were encountered: