LGTM. I probably was waiting for you to get back from leave, and then forgot about it.
Not looking in detail, as this was definitely the intention at the time. You should probably scan through it briefly though.
@nkinkade said:
There are 21 commits in this PR, most of which I don't know the intention without read a lot of history. I'm going to just assume that they are safe to merge since they were probably all code reviewed. We can rollback any change that breaks something. :)
See this closed PR for details on this one:
https://github.com/ndt-project/ndt/pull/219
To summarize:
@gfr10598 said:
@nkinkade said:
This change is