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

Enhance DynamicallyAccessedMembers Attribute #49778

Merged
merged 46 commits into from
Mar 18, 2021

Conversation

LakshanF
Copy link
Contributor

Fixes "Enhance DynamicallyAccessedMembers Attribute to apply to class", #49465, as it was approved

LakshanF and others added 30 commits October 29, 2020 05:18
…iagnostics/DiagnosticSourceEventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
…iagnostics/DiagnosticSourceEventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
…acing/EventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
@dotnet-issue-labeler
Copy link

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@LakshanF LakshanF requested a review from eerhardt March 17, 2021 20:43
@eerhardt
Copy link
Member

When we merge this, let's make sure the 46 commit messages in this PR don't show up in the commit history.

@LakshanF LakshanF merged commit e9d966f into dotnet:main Mar 18, 2021
@LakshanF LakshanF deleted the LinkerAPIChange branch March 18, 2021 16:51
@LakshanF
Copy link
Contributor Author

Merged since the failing check was runtime-staging which is experimental

@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
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.

4 participants