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

Remove AssemblySymbol.RuntimeSupportsNumericIntPtr once we add TargetFramework.Net70 #61235

Closed
jcouv opened this issue May 11, 2022 · 0 comments · Fixed by #64490
Closed

Remove AssemblySymbol.RuntimeSupportsNumericIntPtr once we add TargetFramework.Net70 #61235

jcouv opened this issue May 11, 2022 · 0 comments · Fixed by #64490
Assignees
@jcouv
Labels
Area-Compilers Test Test failures in roslyn-CI
Milestone
17.5

Comments

@jcouv
Copy link
Member

jcouv commented May 11, 2022 •
edited
Loading

The numeric IntPtr tests rely on this test hook because we don't have a target framework with the desired runtime feature flag.
Once we have such a target framework, we can remove the test hook.

Relates to test plan #60578
@jcouv jcouv added this to the C# 11.0 milestone May 11, 2022
@jcouv jcouv self-assigned this May 11, 2022
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Issues and PRs which have not yet been triaged by a lead label May 11, 2022
@jcouv jcouv added Test Test failures in roslyn-CI and removed untriaged Issues and PRs which have not yet been triaged by a lead labels May 12, 2022
@jaredpar jaredpar modified the milestones: C# 11.0, 17.4 Jun 24, 2022
@dibarbet dibarbet mentioned this issue Aug 3, 2022
Closed
@jcouv jcouv modified the milestones: 17.4, 17.5 Sep 30, 2022
@arunchndr arunchndr modified the milestones: 17.5, 17.5 P1 Oct 3, 2022
@jcouv jcouv mentioned this issue Oct 4, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcouv jcouv

Labels
Area-Compilers Test Test failures in roslyn-CI
Projects
None yet
Milestone
17.5
Development

Successfully merging a pull request may close this issue.

Add and use TargetFramework.Net70 jcouv/roslyn
3 participants
@jaredpar @jcouv @arunchndr