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

Suppress Warnings in generated classes #145

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Wavesonics
Copy link

These were mostly a problem with generics

These were mostly a problem with generics
@stephanenicolas
Copy link
Collaborator

@Wavesonics can you make the PR green please ?

@Wavesonics
Copy link
Author

Fun fact: You can use travis to fully compile and iterate on your code...

Anyway, looks green now.

@Wavesonics
Copy link
Author

Any change to get this PR into the next release?

@f2prateek
Copy link
Owner

@Wavesonics looks like the indentation is a bit off in a few places.

@stephanenicolas
Copy link
Collaborator

@Wavesonics can you fix the indentation so that we can merge it ?

@stephanenicolas stephanenicolas added this to the 3.0.0 milestone Dec 2, 2017
@dlemures
Copy link
Collaborator

The PR is targeting an older version of Dart. @Wavesonics can you reproduce the issue you are trying to solve with the version 3.0.1?

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

Successfully merging this pull request may close these issues.

None yet

4 participants