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

Support ConstructorElement #1183

Merged
merged 11 commits into from Sep 28, 2022
Merged

Conversation

AlexV525
Copy link
Contributor

@AlexV525 AlexV525 commented Aug 4, 2022

@AlexV525 AlexV525 marked this pull request as ready for review August 4, 2022 07:52
@AlexV525
Copy link
Contributor Author

cc @kevmoo

@AlexV525 AlexV525 requested a review from kevmoo August 21, 2022 16:09
@kevmoo
Copy link
Collaborator

kevmoo commented Sep 13, 2022

Plan to fix the tests here, @AlexV525 ?

@AlexV525
Copy link
Contributor Author

Plan to fix the tests here, @AlexV525 ?

These analyze failures seem not realted to the PR it self, but with the analyzer's callers. Should I do some rebase?

@kevmoo
Copy link
Collaborator

kevmoo commented Sep 14, 2022

@AlexV525 – yup, you'll want to rebase!

@AlexV525
Copy link
Contributor Author

@AlexV525 – yup, you'll want to rebase!

Done. 😃

@AlexV525
Copy link
Contributor Author

Test has been added to test/src/_json_serializable_test_input.dart, eventually.

@kevmoo
Copy link
Collaborator

kevmoo commented Sep 28, 2022

@AlexV525 – we now have merge conflicts.

@kevmoo kevmoo merged commit aee8507 into google:master Sep 28, 2022
@AlexV525
Copy link
Contributor Author

Thanks, Kevin @kevmoo !

@AlexV525 AlexV525 deleted the support-constructor-element branch September 29, 2022 07:20
@kevmoo
Copy link
Collaborator

kevmoo commented Sep 29, 2022 via email

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

2 participants