-
Notifications
You must be signed in to change notification settings - Fork 5
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
support deprecation #3
Comments
@tmc yes, I plan on pushing deprecated fields to But how would |
The two options that come to mind are parsing a comment or introducing a protobuf extension to allow supplying structured metadata. |
@tmc I already have a few extensions here which I haven't written documentation for yet. I think we can add a free-form directive option. Such as: message X {
string y = 1 {
option (twirpql.options.field) = {
directives = ["@deprecated(reason: whatever)", "@key"]
}
};
} TwirpQL can still support the original deprecated option as well, but it would prioritize the custom inputs |
Do you plan to support field metadata such as isDeprecated/deprecationReason?
The text was updated successfully, but these errors were encountered: