enh(csharp) add Contextual keywords file, args, dynamic, record, required and scoped

[alvinsjoy]

Resolves #4050

Changes

Add new contextual keywords file, args, dynamic, record, required and scoped.

Checklist

• Added markup tests, or they don't apply here because no major changes
• Updated the changelog at CHANGES.md

[github-actions]

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

4 files changed

Total change +41 B

View Changes

file	base	pr	diff
es/core.min.js	8.2 KB	8.2 KB	+1 B
es/highlight.min.js	8.2 KB	8.2 KB	+1 B
es/languages/csharp.min.js	1.68 KB	1.7 KB	+19 B
languages/csharp.min.js	1.69 KB	1.71 KB	+20 B

[github-actions]

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +42 B

View Changes

file	base	pr	diff
es/core.min.js	8.2 KB	8.2 KB	+1 B
es/highlight.min.js	8.2 KB	8.2 KB	+1 B
es/languages/csharp.min.js	1.68 KB	1.7 KB	+19 B
highlight.min.js	8.23 KB	8.23 KB	+1 B
languages/csharp.min.js	1.69 KB	1.71 KB	+20 B

[github-actions]

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +42 B

View Changes

file	base	pr	diff
es/core.min.js	8.2 KB	8.2 KB	+3 B
es/highlight.min.js	8.2 KB	8.2 KB	+3 B
es/languages/csharp.min.js	1.68 KB	1.7 KB	+16 B
highlight.min.js	8.23 KB	8.23 KB	+2 B
languages/csharp.min.js	1.69 KB	1.71 KB	+18 B

[joshgoebel]

Ok removed. But wouldn't we also want to add it to the list of data types?

[alvinsjoy]

I've removed nint from the contextual keywords. When you mentioned a "list of types," were you referring to the built-in keywords? If so nint is already present there. If you were referring to a different list, could you specify which one?

Hi, as I mentioned in the pull request review reply, could you please clarify which list you are referring to as "data types"?

[joshgoebel]

Could be named better, but I'm referring to BUILT_IN_KEYWORDS. If you wanted to rename that to TYPES and scope them all as type that'd be fine also.

[alvinsjoy]

nint was already present in BUILT_IN_KEYWORDS, so should it be kept as it is for now? Are there any other keywords that has to be moved?

[github-actions]

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +31 B

View Changes

file	base	pr	diff
es/core.min.js	8.2 KB	8.2 KB	-1 B
es/highlight.min.js	8.2 KB	8.2 KB	-1 B
es/languages/csharp.min.js	1.68 KB	1.7 KB	+16 B
highlight.min.js	8.23 KB	8.23 KB	-1 B
languages/csharp.min.js	1.69 KB	1.71 KB	+18 B

[github-actions]

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

5 files changed

Total change +31 B

View Changes

file	base	pr	diff
es/core.min.js	8.2 KB	8.2 KB	-1 B
es/highlight.min.js	8.2 KB	8.2 KB	-1 B
es/languages/csharp.min.js	1.68 KB	1.7 KB	+16 B
highlight.min.js	8.23 KB	8.23 KB	-1 B
languages/csharp.min.js	1.69 KB	1.71 KB	+18 B

[joshgoebel]

THanks so much!