docs(ast): fix comment of ClassElement::r#static

[kermanx]

The comment says: Returns true if this ClassElement is a static block or has a static modifier.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[codspeed-hq]

CodSpeed Performance Report

Merging #6731 will not alter performance

_{Comparing KermanX:fix/class-element-static (3a4117f) with main (a7dd5aa)}

Summary

✅ 30 untouched benchmarks

[Dunqing]

Thanks!

Wrong

[Dunqing]

Wait, I just realized this comment was added later and it's wrong.

[DonIsaac]

Yeah it returns true for static properties. Static blocks aren't properties

[kermanx]

I've reverted the code change and updated the comment :)

[overlookmotel]

Sorry to be a pain, but I don't think this is quite right either, because this suggests it has to be a property, not a method.

I suggest let's clear this up for once and for all!

/// Returns `true` if this [`ClassElement`] has a static modifier.
/// 
/// Note: Class static blocks do not have a "modifier", as there is no non-static equivalent.
/// Therefore, returns `false` for static blocks.
///
/// The following all return `true`:
/// ```ts
/// class {
///   static prop = 1;
///   static method() {}
///   static #private = 2;
///   static #privateMethod() {}
///   static accessor accessorProp = 3;
///   static accessor #privateAccessorProp = 4;
/// }
/// ```

[overlookmotel]

I'm too late. Merged while I was writing! Will make a follow-up PR.