Skip to content
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

Annotation (processing) improvements #758

Open
Citymonstret opened this issue Jul 27, 2024 · 0 comments
Open

Annotation (processing) improvements #758

Citymonstret opened this issue Jul 27, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@Citymonstret
Copy link
Member

During the command method validation we should attempt to validate nullability, with some support for common nullability annotations.

If a parameter is required we should output a warning if it's marked as nullable. If it's optional without a default value, then we should probably error if it's marked as non-nullable.

We ought to validate whether primitive values are optional without defaults, as this will fail at runtime.

It'd also be good if we added some support for third-party annotations at runtime. Such as existing range annotations, and possibly even existing pattern annotations (for regex), etc.

We should properly mark our annotations as inherited and documented, and improve our support for meta-annotations and annotation composition.

None of this would break API and can be done after the 2.0 release.

@Citymonstret Citymonstret added the enhancement New feature or request label Jul 27, 2024
@Citymonstret Citymonstret self-assigned this Jul 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant