Skip to content

Fixes #1372. Expected error message text fixed #1373

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 1 commit into from
Jul 25, 2022

Conversation

sgrekhov
Copy link
Contributor

No description provided.

Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

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

LGTM!

@eernstg eernstg merged commit b76040d into dart-lang:master Jul 25, 2022
@eernstg
Copy link
Member

eernstg commented Jul 25, 2022

A similar PR is upcoming for the pre-nnbd branch?

@sgrekhov
Copy link
Contributor Author

@eernstg Hmm. No. Does Horizontal inference feature affect pre-nnbd as well? If so I'll add these tests to pre-nnbd branch.

@eernstg
Copy link
Member

eernstg commented Jul 25, 2022

Oh, maybe I just misremember, but I thought that I saw a failure on both branches about this extra period on the error message, and I didn't notice that there was a connection to horizontal inference. Let me try to find the failure log...

@eernstg
Copy link
Member

eernstg commented Jul 25, 2022

I think it must have been a completely different test, this one is in the Horizontal-inference directory (and I did not see that, I just thought I could remember that I saw this "period failure" twice ;-).

@eernstg
Copy link
Member

eernstg commented Jul 25, 2022

OK, the log is here, I was just misremembering the situation. So we're done, thanks!

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jul 25, 2022
2022-07-25 [email protected] Fixes dart-lang/co19#1372. Expected error message text fixed (dart-lang/co19#1373)
2022-07-22 [email protected] Fixes dart-lang/co19#1356. Add analyzer errors on references to not included library parts (dart-lang/co19#1360)
2022-07-22 [email protected] Fixes dart-lang/co19#1354. Horizontal inference tests added (dart-lang/co19#1358)
2022-07-19 [email protected] Fixes dart-lang/co19#1367. Missed errors expectations added for CFE (dart-lang/co19#1371)
2022-07-14 [email protected] Fix quoting in spec parser command line (dart-lang/co19#1369)
2022-07-14 [email protected] Ignore deleted files in spec parser action (dart-lang/co19#1368)
2022-07-14 [email protected] Pin GitHub actions and setup dependabot (dart-lang/co19#1365)

Change-Id: Ib57b9c95b1a50c20a3716383cd6839f65950bac1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/252603
Commit-Queue: Erik Ernst <[email protected]>
Reviewed-by: Alexander Thomas <[email protected]>
sgrekhov added a commit to sgrekhov/co19 that referenced this pull request Aug 4, 2022
@sgrekhov sgrekhov deleted the co19-1372 branch September 2, 2022 06:43
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.

2 participants