GeezerVison

I feel this one is not important to be put here. For example, some developers might always make sure all local variables are initialized before use, which is fine and won't actually cause any overhead -- IL code will have the assignment but JIT will be smart enough to not do the assignment.

Instead, I think we should mention the Interlocked type here, which I saw in Managed Threading Best Practices:
Consider using the Interlocked class instead of the lock statement to atomically change simple states. The Interlocked class provides better performance for updates that must be atomic.

Will add Interlocked.

We should have a consensus about initializations - it seems your positions is different from the Paul Higinbotham (@PaulHigin) position.

I think null/zero initializations for local variables doesn't matter perf-wise. Personally, I prefer to always initialize local variables before use, but that's not something should be enforced to everyone.
For type fields, I saw some are initialized to 0, but rarely see people initialize them to null, but either way, I won't pick it in code review.
For auto properties, I never saw people initialize it to null or zero (maybe there are but rare).

Overall, I think we should just remove this item.

Removed.

Let's not say follow because some of those guidelines are contradicting with our coding convention. For example, the naming guideline in the first link below says "do not use "g_" or "s_" to indicate static fields", while we require static fields to start with s_ or t_.

I think it's better to say: "Here are some useful links for your reference:"

Fixed.

This one seems to focus on full .NET instead of .NET Core, so it mainly talks about things related to WebRequest and WebResponse.

I added just for WebRequest and WebResponse.
Remove?

WebRequest and WebResponse are mostly deprecated, and HttpClient is the way to go. So I think we can remove this link.

Removed.

Please use capital P for practices and capital E for exceptions to be consistent with others.

Fixed.

I have been very confused about when to use InvariantCulture vs. Ordinal, and I believe there are many other developers like me :) I think this one is very useful.

Please use capital G for guidelines.

Fixed.

I think null/zero initializations for local variables doesn't matter perf-wise. Personally, I prefer to always initialize local variables before use, but that's not something should be enforced to everyone.
For type fields, I saw some are initialized to 0, but rarely see people initialize them to null, but either way, I won't pick it in code review.
For auto properties, I never saw people initialize it to null or zero (maybe there are but rare).

Overall, I think we should just remove this item.

WebRequest and WebResponse are mostly deprecated, and HttpClient is the way to go. So I think we can remove this link.