Schuyler Eldridge

CD added here: https://github.com/chipsalliance/firrtl-spec/commit/1420ea33f88cab2e2e5f6301d1ce125b2919255f

By my count there are ~8k code samples on GitHub as gleaned from the following search: https://github.com/search?p=10&q=extension%3Afir+NOT+flang+NOT+func+NOT+%22RUN%22+-user%3AKosinskiLab+-user%3Alukezya+-user%3ALourencoPonces+-user%3ALesible+-user%3Ajoaomiguelcaldeira&type=Code This is slightly complicated as `.fir` shares a file extension with FLANG IR.

Checking for the version string / FIRRTL in it would enable a trivial heuristic to differentiate the two. Great thinking.

Checking for `FIRRTL` or `circuit` sounds great. 👍

Fixed with https://github.com/github-linguist/linguist/pull/6848.

Updated the above. This was missing an additional restriction on equivalent widths. Fair point about `Uint

I am of the opinion that we should leave in the cover message, rename it to "comment", disallow format strings in comments (it must be a pure string), and specify...

I'm not terribly against splitting this up, though I actually prefer it as one file for the purposes of easier searching when editing locally. If we want to do this,...

I'm intending to roll back the scope of this to narrowly define the `String` type, define a `String` literal (which already implicitly exists), and make it clear that this is...

This is now relaxed to only add a `String` type and describe a `String` literal. This is narrowly a patch change as opposed to actually defining any new behavior. Thanks...