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

Merge "generic attributes" to main #55577

Merged
merged 16 commits into from
Aug 13, 2021
Merged

Conversation

RikkiGibson
Copy link
Contributor

Related to #36285

We're shipping this feature in .NET 6 under the "preview" LangVersion.

We don't have an EnC test here but honestly I would like if I could wait for #54891 to be merged before writing one :). Will do this before "promoting" this feature out of preview. When I tried to verify manually I got a rude edit message. I think I was on a pretty recent .NET 6, but not certain now, so am also curious how I can determine which runtimes support editing custom attributes during EnC.

jcouv and others added 16 commits February 23, 2021 17:03
Refresh generic-attributes branch with latest bits from master
Co-authored-by: Julien Couvreur <julien.couvreur@gmail.com>
Co-authored-by: Rikki Gibson <rigibson@microsoft.com>
…n into merges/main-to-features/generic-attributes
…-attributes

Merges/main to features/generic attributes
…-attributes

Merges/main to features/generic attributes
…-attributes

Merges/main to features/generic attributes
@RikkiGibson RikkiGibson requested a review from a team as a code owner August 12, 2021 20:08
@RikkiGibson
Copy link
Contributor Author

I've been advised to ignore the lsp-CI failure.

@RikkiGibson RikkiGibson merged commit 675a588 into main Aug 13, 2021
@ghost ghost added this to the Next milestone Aug 13, 2021
@RikkiGibson
Copy link
Contributor Author

We did it @AviAvni 😄. This will be available in the "preview" LangVersion in dev17-preview4 and hopefully will release "stable" in .NET 7 / C# 11. Thanks for your help with this feature.

@RikkiGibson RikkiGibson deleted the features/generic-attributes branch August 13, 2021 00:31
@AviAvni
Copy link
Contributor

AviAvni commented Aug 13, 2021

@RikkiGibson Thank you very much

@davidwengier
Copy link
Member

I would like if I could wait for #54891 to be merged before writing one

The good news is, this is merged!
The bad news is, this is merged.

@RikkiGibson
Copy link
Contributor Author

🙈

@dibarbet dibarbet modified the milestones: Next, 17.0.P4 Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants