Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve directory attribute refactor - Break up PR #18161 - Part 2 #18196

Merged

Conversation

KevinRansom
Copy link
Member

@KevinRansom KevinRansom commented Jan 3, 2025

Contains all of the modified test cases that are not in tests\FSharp.Compiler.ComponentTests\EmittedIL.

Also deletes:

tests\FSharp.Compiler.ComponentTests\AttributeUsage.fs
tests\FSharp.Compiler.ComponentTests\OnOverridesAndIFaceImpl.fs

The tests were duplicated and had been updated in:

tests\FSharp.Compiler.ComponentTests\Conformance\BasicGrammarElements\CustomAttributes\AttributeUsage\AttributeUsage.fs
tests\FSharp.Compiler.ComponentTests\Conformance\BasicGrammarElements\AccessibilityAnnotations\OnOverridesAndIFaceImpl\OnOverridesAndIFaceImpl.fs

Moved MethodResolution.fs into a subdirectory and added baselines.

Note it also contains the commit that updates the test framework from part 1

/cc @T-Gro

@KevinRansom KevinRansom requested a review from a team as a code owner January 3, 2025 02:11
Copy link
Contributor

github-actions bot commented Jan 3, 2025

✅ No release notes required

@KevinRansom KevinRansom force-pushed the ImproveDirectoryAttribute.sometests branch from 13c8b3b to bc81456 Compare January 3, 2025 02:14
Copy link
Member

@T-Gro T-Gro left a comment

Choose a reason for hiding this comment

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

(Approving the part which adds on Part 1, merge only after Part 1 gets merged)

@KevinRansom KevinRansom force-pushed the ImproveDirectoryAttribute.sometests branch from 75153dd to 400a8fa Compare January 7, 2025 17:35
@KevinRansom
Copy link
Member Author

image

@majocha
Copy link
Contributor

majocha commented Jan 7, 2025

@psfinaki psfinaki merged commit a310d1d into dotnet:main Jan 8, 2025
33 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants