[Version 9.0] Feature support for nullable attributes carried over from V8#1221

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

BillWagner wants to merge4 commits intodraft-v9

base:draft-v9

Choose a base branch

fromAdd-v9-nullable-attributes

Draft

[Version 9.0] Feature support for nullable attributes carried over from V8#1221

BillWagner wants to merge4 commits intodraft-v9fromAdd-v9-nullable-attributes

Conversation

Copy link

Member

BillWagner commentedNov 22, 2024

These two attributes were mistakenly added in#1191. They were removed in#1218

This PR adds them back into thedraft-v9 branch, where they are valid.

Restore nullable attributes added in 9

098d779

These two attributes were mistakenly added in#1191.  They were removed in#1218This PR adds them back into the `draft-v9` branch, where they are valid.

BillWagner added this to theC# 9.0 milestone

Nov 22, 2024

fix link

75e3338

Copy link

Contributor

RexJaeschke commentedJun 16, 2025

Originally, I had these two attributes spec'd for V8, but they have been moved to V9. I just proofed the changes made by this PR, and they look good to me.

However, I seem to recall Bill telling me these should be marked as "conditionally normative." That is, support for them is optional, but if either is included, they must have the syntax/semantics, as specified. Should they be a combined package? That is, if one is implemented then so too should the other?