Skip to content

Support FutureOr<String> for Generator{ForAnnotation} #263

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

Merged
merged 4 commits into from
Jul 27, 2017

Conversation

jakemac53
Copy link
Contributor

closes #250

@jakemac53 jakemac53 added the type-enhancement A request for a change that isn't a bug label Jul 27, 2017
@jakemac53 jakemac53 requested a review from natebosch July 27, 2017 15:13
@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored by someone other than the pull request submitter. We need to confirm that they're okay with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@matanlurey matanlurey changed the title update generate and generateForAnnotation to return a FutureOr instead of a Future Support FutureOr<String> for Generator{ForAnnotation} Jul 27, 2017
@matanlurey matanlurey merged commit 9e70efa into master Jul 27, 2017
@matanlurey matanlurey deleted the use-future-or branch July 27, 2017 20:04
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…ce_gen#263)

* update generate and generateForAnnotation to return a FutureOr instead of a Future

* update pubspec/changelog
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…ce_gen#263)

* update generate and generateForAnnotation to return a FutureOr instead of a Future

* update pubspec/changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Return type of generate method in Generator could be FutureOr
4 participants