- User Since
- Jul 17 2020, 11:08 AM (19 w, 2 d)
Wed, Nov 11
I've added some documentation that hopefully highlights the intent behind the change. The way that we generate rewrite implementations for ops (i.e. the build() call that we use) means that we can omit some of the attributes if they're marked optional and everything should "just work". The tests that were added will hopefully catch this if anything fails in the future. PTAL (and let me know if I'm missing something egregious here!)
Add some documentation and refine the tests a bit further.
Fix some tests to use actual required args and add failure tests
Fix cases where we may have location directives
Oct 20 2020
Fix clang-format to use camel case instead of snake case.