Skip to content

Conversation

@alorenzen
Copy link
Contributor

Extract matchTypes from matchAnnotation so that it can be used in cases where the type comes from a different source (like a DartObject instance).

Extract matchTypes from matchAnnotation so that it can be used in cases where the type comes from a different source (like a DartObject instance).
Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the changelog for 0.5.3 w/ this addition.

Also: are you using it outside of this package? If so, we should discuss where – since this is not part of the official source_gen api. It's implementation.

return matchTypes(annotationType, annotationValueType);
}

bool matchTypes(Type annotationType, ParameterizedType annotationValueType) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add a Doc comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@natebosch natebosch self-requested a review January 6, 2017 01:02
@natebosch
Copy link
Member

I'm on board with the change. Will wait to merge it in until I get a chance to sync with @kevmoo

@natebosch natebosch merged commit 362b9e3 into dart-lang:master Jan 9, 2017
matanlurey pushed a commit to angulardart/angular that referenced this pull request Jan 18, 2017
See dart-lang/source_gen#116 for the corresponding source_gen PR.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=144028180
matanlurey pushed a commit to angulardart/angular that referenced this pull request Jan 18, 2017
See dart-lang/source_gen#116 for the corresponding source_gen PR.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=144028180
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
Extract matchTypes from matchAnnotation so that it can be used in cases where the type comes from a different source (like a DartObject instance).
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
Extract matchTypes from matchAnnotation so that it can be used in cases where the type comes from a different source (like a DartObject instance).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants