Skip to content

Remove feature exclude xml tests - #89111

Merged
marek-safar merged 1 commit into
dotnet:mainfrom
Unity-Technologies:linker-remove-feature-exclude-xml-tests
Jul 21, 2023
Merged

Remove feature exclude xml tests#89111
marek-safar merged 1 commit into
dotnet:mainfrom
Unity-Technologies:linker-remove-feature-exclude-xml-tests

Conversation

@mrvoorhe

Copy link
Copy Markdown
Contributor

illink dropped the exclude feature functionality, so will we.

I believe Unity was the last user of these tests so it's safe to delete them now.

illink dropped the exclude feature functionality, so will we.

I believe Unity was the last user of these tests so it's safe to delete them now.
@mrvoorhe
mrvoorhe requested a review from marek-safar as a code owner July 18, 2023 17:24
@ghost ghost added linkable-framework Issues associated with delivering a linker friendly framework area-Tools-ILLink .NET linker development as well as trimming analyzers community-contribution Indicates that the PR has been added by a community member labels Jul 18, 2023
@ghost

ghost commented Jul 18, 2023

Copy link
Copy Markdown

Tagging subscribers to 'linkable-framework': @eerhardt, @vitek-karas, @LakshanF, @sbomer, @joperezr, @marek-safar
See info in area-owners.md if you want to be subscribed.

Issue Details

illink dropped the exclude feature functionality, so will we.

I believe Unity was the last user of these tests so it's safe to delete them now.

Author: mrvoorhe
Assignees: -
Labels:

linkable-framework

Milestone: -

@marek-safar
marek-safar merged commit 04c0bd7 into dotnet:main Jul 21, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Aug 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Tools-ILLink .NET linker development as well as trimming analyzers community-contribution Indicates that the PR has been added by a community member linkable-framework Issues associated with delivering a linker friendly framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants