Austin Henriksen
Austin Henriksen
Also, to disavow myself of my biases here: IMO, 4 space indentation is objectively easier to read than 2 spaces spoken as someone who is currently editing large chunks of...
I agree that we shouldn't spend time tweaking and modifying the defaults. My thinking was just switching which format-style we were using, but keeping it's defaults intact. It looks like...
> Unfortunately these are not compatible Like the 2 tools don't format the code in the same way? If so, that's indeed pretty lame. Tomorrow I plan to at least...
This was implemented in #2734.
It was on the edge of being too hacky for me, so I'll unhackify it, even if it's a little redundant. _If_ we ever add proper metadata validation in the...
> I thought our idea was to use the word tag going forward to limit confusion That's going to be a tall order. Updating all our code/docs to use `tag`....
Yes, you're right. I'll look around the code for any references to `tag`. And remove those too, if everyone is okay with this going forward in the first place.
We had a separate issue open for removing the 'tagged's that snuck in: https://github.com/zeroc-ice/ice/issues/1623 I'm posting it here to link the two, and just to quote from it: `however, the...
I altered this PR to also fix any mentions of the `tagged` keyword from `icerpc`. I think it's pretty trivial, so I'm not remarking reviewers, but feel free to re-re-view...
1) The broader point of my comment was: "an error message which requires niche domain knowledge or reading a spec isn't a great error message". But this is a niche...