Andrew Azores

Worth looking into https://github.com/koalaman/shellcheck . You could also set this up for CI on each PR using GitHub Actions/workflows to ensure that all accepted code contributions are properly portable across...

^ https://github.com/marketplace/actions/run-shellcheck-with-reviewdog and/or https://github.com/marketplace/actions/shellcheck

Ideally this is implemented using the same endpoint - it's the same HTTP verb and on the same resource (same path), so this implies it should be implemented in the...

> When do I know if I should update the v1 vs. v2 handlers or both? I guess that kind of decision falls on me :-) In the case of...

Adding path-specific behaviours to the `WebServer` is not desirable. The `RequestHandler` system we have is an abstraction designed specifically so that the `WebServer` class is as generic as possible and...

Right, good catch. We I don't think we want to format the data as both HTML and JSON up-front, since as you say that's wasting resources (CPU time as well...

Closed by #1032 / #1028

Simply checking that the environment variable is set is probably good enough, at least for now. I'm not sure it would necessarily be worth trying to perform some pre-verification that...

Jie and I have talked about that very briefly, but we're still not completely sure what it should look like. Some of the rules probably do correspond very well with...

Could this be caused by #758? Only thing that went in recently that seems like it might affect this.