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.

Sign up for GitHub

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

Jump to bottom

#2559. Replace augmentation libraries by parts #2848

Merged
merged 2 commits into from
Sep 9, 2024
Merged

#2559. Replace augmentation libraries by parts #2848

merged 2 commits into from
Sep 9, 2024

Conversation

sgrekhov
Copy link
Contributor

@sgrekhov sgrekhov commented Sep 6, 2024

No description provided.

eernstg
eernstg reviewed Sep 6, 2024
View reviewed changes
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.

Looks good! I noted a couple of locations where it seems to have been transformed by a search and replace, but the result isn't quite natural.

Also, I get about 20 results on a grep for 'library augmentation'. That would probably become 'part' in most cases.

LanguageFeatures/Augmentation-libraries/augmenting_declarations_A02_t02.dart Outdated Show resolved Hide resolved
LanguageFeatures/Augmentation-libraries/defining_augmentation_A02_t06.dart Outdated Show resolved Hide resolved
LanguageFeatures/Augmentation-libraries/defining_augmentation_A07_t01.dart Outdated Show resolved Hide resolved
LanguageFeatures/Augmentation-libraries/defining_augmentation_A07_t01.dart Show resolved Hide resolved
LanguageFeatures/Augmentation-libraries/defining_augmentation_A08_t01.dart Outdated Show resolved Hide resolved
@sgrekhov
Copy link
Contributor Author

sgrekhov commented Sep 9, 2024

Updated.
There still are 'library augmentation' strings in these tests but all of them are part of obsolete assertions. I'm going to update defining_augmentations_* tests to fix all of these assertions (there is no more that part in the spec) but in a separate PR if you don't mind. PTAL.

@eernstg eernstg merged commit 1c9cbaf into dart-lang:master Sep 9, 2024
1 of 2 checks passed
@eernstg
Copy link
Member

eernstg commented Sep 9, 2024

OK, thanks!

sgrekhov added a commit to sgrekhov/co19 that referenced this pull request Sep 9, 2024
@sgrekhov
dart-lang#2559. Replace augmentation libraries by parts (dart-lang#2848)
a81d97c 
Replace augmentation libraries by parts. Note that the grammar hasn't yet been updated to allow for import directives in part files (it will be updated).
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Sep 10, 2024
@sgrekhov
[co19] Roll co19 to a572236bc7d760dc986fd34abe6d0b8ae56254d7
ab23d7a 
2024-09-10 sgrekhov22@gmail.com Fixes dart-lang/co19#2852. Fix the new roll failures (dart-lang/co19#2853)
2024-09-09 sgrekhov22@gmail.com dart-lang/co19#2825. Add scope tests. Part 2. (dart-lang/co19#2847)
2024-09-09 sgrekhov22@gmail.com Fixes dart-lang/co19#2849. Fix new roll failures (dart-lang/co19#2850)
2024-09-09 sgrekhov22@gmail.com dart-lang/co19#2559. Replace augmentation libraries by parts (dart-lang/co19#2848)
2024-09-05 sgrekhov22@gmail.com dart-lang/co19#2825. Add scope tests. Part 1. (dart-lang/co19#2845)
2024-09-04 sgrekhov22@gmail.com dart-lang/co19#2398. Update TouchEvent tests to not fail on platforms where it is not defined (dart-lang/co19#2844)
2024-09-03 sgrekhov22@gmail.com dart-lang/co19#2825. Add import inheritance and grammar tests (dart-lang/co19#2843)
2024-09-02 sgrekhov22@gmail.com dart-lang/co19#2559. Add augmenting members tests. Part 2 (dart-lang/co19#2799)
2024-09-02 sgrekhov22@gmail.com dart-lang/co19#2825. Add enhanced parts tests for top-level declarations and ownership (dart-lang/co19#2842)
2024-09-01 49699333+dependabot[bot]@users.noreply.github.com Bump actions/setup-java from 4.2.1 to 4.2.2 in the github-actions group (dart-lang/co19#2841)
2024-08-30 sgrekhov22@gmail.com dart-lang/co19#2559. Add extension types to augmenting types tests (dart-lang/co19#2839)

Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try
Change-Id: I26f83579708b877918b5a0a904f8172aa1cf4724
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/384480
Reviewed-by: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eernstg eernstg eernstg approved these changes

@chloestefantsova chloestefantsova Awaiting requested review from chloestefantsova

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sgrekhov @eernstg