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

Spelling tests #50920

Closed
wants to merge 16 commits into from
Closed

Spelling tests #50920

wants to merge 16 commits into from

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Jan 6, 2023

This change is scoped to tests with any files that happened to be changed in those commits per #50754, it's <65 files.

The meta warns that test changes may require some magic.

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@eernstg
Copy link
Member

eernstg commented Jan 6, 2023

@athomas
Copy link
Member

athomas commented Jan 19, 2023

Mostly LGTM, contains one long line, one definitely unsafe change that is probably best broken out into a separate PR, and one potentially unsafe change. See https://dart-review.googlesource.com/c/sdk/+/278535 for details.

Note: The unsafe change actually enabled a test that now longer needs to be skipped. So the two lines can just be deleted rather than modified. Alternatively, they can be reverted because the file will be deleted later this year.

@athomas athomas self-requested a review January 19, 2023 07:42
@jsoref jsoref deleted the spelling-tests branch January 19, 2023 17:00
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.

3 participants