lemnos / tt

A terminal based typing test.
MIT License
737 stars 49 forks source link

Marking `go` as a dependency #15

Closed tusqasi closed 2 years ago

tusqasi commented 2 years ago

Makes it easier to follow along the readme

tusqasi commented 2 years ago

Sorry if i am spamming PRs, but kinda don't know how this works. Let me know if i messed something up.

lemnos commented 2 years ago

No, I am just horrible at responding to PRs :P. My apologies. This looks fine, but can you remove the apt command or add a comment suggesting it is debian specific?

tusqasi commented 2 years ago

How about now?

lemnos commented 2 years ago

Sorry for the prolonged response. This looks fine, but can you squash these into a single commit? It helps keep the history clean.

tusqasi commented 2 years ago

Is this correct? I don't exactly know if that was correctly done.
Hope that was fine
😅

lemnos commented 2 years ago

The goal is to have a single commit. There are currently still three separate ones. You can use git rebase -i to squash the commits of interest. This useful.

lemnos commented 2 years ago

I've gone ahead and merged this anyway. Just in time for 2022 ;).

Happy new year.