Max Korbel

Good idea, thank you for filing!

This is a cool tool, good idea! What are your thoughts on _when_ this check should be run? Suppose new rules show up and start blocking all open PRs on...

> Consider adding a async-reset clk-div testcase. Added in latest commit, good idea, thanks

Need to add more testing around chains of clock dividers, which are broken with current implementation in this PR. Likely solution is to use topological sort to pick the order...

Something like this has actually been on my mind for another purpose as well: easier debug and hooks for debug tools. I was considering if there should be a way...

Also, think about flushing

Also add for `flop`

Should have either asyncReset, reset, or no reset, but not both. Perhaps `flop(clk, a, asyncReset: reset`

Different cases to consider: - Generate a string with all modules in it (already supported), return as `String` - Generate a string per-module (supported via `SynthBuilder` and `SynthResult`), return as...