175 karma · joined February 19, 2022
The per-request model was pretty insane.
Just read the “outcome” for Warp:
> … and became the most popular modern terminal. Login removed, telemetry made optional — every criticism addressed.
Insane
That seems better as long as you can keep it standard across the team. I don’t usually check each commit when reviewing since frequent iterative commits mean folks change their mind and I’d review already removed logic when looking at early commits.
I’ve been scraping by on basic git usage so didn’t know about fix-up commits, that’s excellent.
You can run a python script to track the focused window every few seconds to identify what’s stealing focus.
My history ends up being: - add feature x - linting - add e2e tests - formatting - additional comments for feature - fix broken test (ci caught this) - update README for new feature - linting
With a squash it can boil down to just “added feature x” with smaller changes inside the description.