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.

• docs(semantic): document SymbolTable #5978 👈
• test(semantic): add comprehensive regression test suite #5976
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on Graphite

CodSpeed Performance Report

Merging #5978 will not alter performance

_{Comparing don/09-22-docs_semantic_document_symboltable_ (c8f1f61) with don/09-22-test_semantic_add_comprehensive_regression_test_suite (15b6b3b)}

Summary

✅ 29 untouched benchmarks

Could you please move this into its own stack, rather than stacked on top of #5976 so can get it merged ASAP? #5976 may take a while to review.

Apart from the typo which @daflyinbed spotted, this looks good to me.

Just a note: I would have preferred if this PR had been split in 2 for (1) AST type docs and (2) SymbolTable docs. They wouldn't even need to be stacked on top of each other as they're independent changes. No need to split it up now, just saying for next time...

Re-created in #5998