Skip to content

Improved typing throughout the Parse library #570

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

Closed
nstrelow opened this issue Mar 6, 2021 · 4 comments
Closed

Improved typing throughout the Parse library #570

nstrelow opened this issue Mar 6, 2021 · 4 comments

Comments

@nstrelow
Copy link
Contributor

nstrelow commented Mar 6, 2021

I am currently getting frustrated fixing lint errors.

Stuff like
List results; without a type
static Future<dynamic> currentUser( returning dynamic instead of a ParseUser

also the custom ParseError which could be replaced by a dart Error or at least extend dart Error.

This would a HUGE breaking change, but it would definitely improve usability in the long run.

@nstrelow nstrelow changed the title Improved typing through the Parse library Improved typing throughout the Parse library Mar 6, 2021
@nstrelow
Copy link
Contributor Author

nstrelow commented Mar 6, 2021

How are you people actually handling the current typing?
Using a lot of as XXX ?

I was avoiding that, because of possible performance problems in web: dart-lang/sdk#57894

But I am not sure if that is still a problem

@fischerscode
Copy link
Contributor

I am using the custom objects and many as.
Moving to generics might be an idea, although this would require quite some work.

@nstrelow
Copy link
Contributor Author

nstrelow commented Mar 8, 2021

I do the same.
Generics would be the better option. In some future with endless time, I might attempt it 🙈

@RodrigoSMarques
Copy link
Contributor

Closed. No activity in the last 14 days.
If necessary open again.

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

No branches or pull requests

3 participants