Steven R. Loomis

Results 747 comments of Steven R. Loomis

OK. Whew. We still pass, even with lots of merging. Now back to refactoring work!

> CompilerCallback I'm thinking now that the import / implied should also be done in a CompilerCallback

@mcdurdin OK PTAL once again. Shuffled things around. #8106 is now the parent of this instead of the middle of the pile. I'll use message subclasses in a subsequent PR.

> Oh, please don't merge approved PRs into unapproved PRs -- it makes the review process quite horrible 😒 I hear you, but ? I don't think I did that… ...

> > I hear you, but ? I don't think I did that…  > > I merged an unreviewed PR here ( #8107 ) which had a refactoring that addressed...

:+1: on asking to clear up confusion, thanks! > I think a note (the same note) in both PRs would be helpful in this case. sounds good… > And sometimes...

@mcdurdin this may be moot, now that we "consume all" forms basically?