-
Notifications
You must be signed in to change notification settings - Fork 47
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
Stats need to mark up which members are required #304
Comments
This is entirely orthogonal to the question of whether they're mandatory-to-implement. |
@alvestrand Can we get a PR to close this issue since it is blocking issuance of the CR? |
I think we can safely remove the CR-blocker label. Doing this at a later stage is not a breaking change, it's just improved documentation. |
alvestrand
added a commit
that referenced
this issue
Apr 21, 2020
alvestrand
added a commit
that referenced
this issue
Apr 23, 2020
alvestrand
added a commit
that referenced
this issue
Apr 23, 2020
* Add "required" specifiers within dictionaries Fixes #304
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some fields are needed for the stats to make any sense at all. Others may or may not be present, depending on the state of the connection or the quality of the implementation.
Those that are absolutely necessary should be marked up with "required" in the IDL.
The text was updated successfully, but these errors were encountered: