matijsvzuijlen

In GitHub and vim at least you can dig past those commits to see earlier relevant commits. I'm sure there are more tools that allow this.

This is a nice idea but a lot of the OSM tiles show as blocked due to lack of a Referer.

Thanks, that makes sense. I actually just parsed aes as related to encryption without even considering how weird that was given the contect.

Yes, I will do that!

Why are all the parameters full words except 'labs'? I find it jarring because to me 'lab' is short for laboratory, not label.

What made you decide to squash when merging instead of leaving the commits in the history so you can always bisect?

I worked on a project that used haml, and merge conflicts in the haml files were very hard to resolve due to the lack of any nesting info besides indentation.

I think it's worse than, e.g., typical yaml files because the nesting is deeper.

If you already know k8s, this is probably true. If you don't it's hard to know what bits you need, and need to learn about, to get something simple set up.

I can understand requiring 100% test coverage, but it seems to me that requiring a test file for every file is preventing your team from doing useful refactoring.

What made your team decide on that rule? Could your team decide to drop it since it hinders improving the design of your code?

What makes you think TDD assumes that? It sounds like the complete opposite of what TDD is about.

In such cases, I generally checkout main, do the side quest, and rebase my original branch on top of it. Then, I can just continue without waiting for the side quest to be merged. Depending on the situation, I may make a separate PR for the side quest, but I don't have to wait for it to get merged.

Alternatively, if you don't like rebasing, you can merge the side quest branch into your project branch instead.

Either way, you don't have to wait for the side quest to get merged.

The idea is that --amend would be used before pushing the commit that it amends, and so before a reviewer ever sees the original commit.

Linking to the abstract instead would have been nice.