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

[NativeAot] Bring up a dozen or so libraries tests #73363

Merged
merged 3 commits into from
Aug 5, 2022

Conversation

MichalStrehovsky
Copy link
Member

This time found some product issues and fixed them.

Cc @dotnet/ilc-contrib

This time found some product issues and fixed them.
@ghost
Copy link

ghost commented Aug 4, 2022

Tagging subscribers to this area: @dotnet/area-meta
See info in area-owners.md if you want to be subscribed.

Issue Details

This time found some product issues and fixed them.

Cc @dotnet/ilc-contrib

Author: MichalStrehovsky
Assignees: -
Labels:

area-Meta

Milestone: -

@MichalStrehovsky
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MichalStrehovsky
Copy link
Member Author

MichalStrehovsky commented Aug 4, 2022

Sigh, some of this will need tweaking because #73362 is apparently fixing the same issue I ran into in a better way.

The comment applies to the changes I have in #73283. My head is spinning from all these tests. Done for the day.

Copy link
Member

@vitek-karas vitek-karas 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 to me

@@ -82,6 +84,7 @@ public static IEnumerable<object[]> CultureResourceData()

[Theory]
[MemberData(nameof(CultureResourceData))]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155", typeof(PlatformDetection), nameof(PlatformDetection.IsNativeAot))]
Copy link
Member

Choose a reason for hiding this comment

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

This issue has a new home: #69919

It would be nice to update the existing references of https://github.com/dotnet/runtimelab/issues/155 to this one.

Copy link
Member Author

Choose a reason for hiding this comment

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

We can do it in one go later. 155 is a number that I don't need to look up because I remember it. 69919 is harder to remember. I often need it during bringup and I want any tiny help that makes the bringup easier. It's very tedious and repetitive.

@MichalStrehovsky
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MichalStrehovsky MichalStrehovsky merged commit b13e6ef into dotnet:main Aug 5, 2022
@MichalStrehovsky MichalStrehovsky deleted the moretests6 branch August 5, 2022 05:29
@ghost ghost locked as resolved and limited conversation to collaborators Sep 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants