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

Mark required parameters throughout API with @required #176

Open
bkonyi opened this issue Apr 13, 2020 · 1 comment
Open

Mark required parameters throughout API with @required #176

bkonyi opened this issue Apr 13, 2020 · 1 comment

Comments

@bkonyi
Copy link
Member

bkonyi commented Apr 13, 2020

In some situations it's unclear which parameters are required, particularly for the OAuth flow. We should investigate whether or not there would be any benefit of marking parameters as @required, although this may be handled after non-nullability lands in Dart.

@misterfourtytwo
Copy link

misterfourtytwo commented Apr 1, 2021

i guess it should be closed now after migration to null-safety, where required is kinda mandatory now.
or i could do this one here on coming weekend, if this task is still necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants