search

exercism / ocaml

Exercism exercises in OCaml.
https://exercism.org/tracks/ocaml
MIT License
93 stars 50 forks source link

Bob: Remove hint #367

Closed sshine closed 4 years ago

sshine commented 4 years ago

This fixes #365. See this issue for discussion.

This PR should be rebased after #366.

Note: I tried to branch off from the feature branch used in #366 hoping that this one could be immediately rebased after it without having to resolve merge conflicts. This is an experiment and may turn out to be a bad strategy.

sshine commented 4 years ago

@marionebl: This showcases that if you have multiple feature branches with one commit in each, and you want to fast-line them, you can branch one off another. But admittedly, if the first feature branch in a chain contains a lot of commits, this will flood the second PR overview, and if the second PR contains more than one commit, it will be very hard to decipher which commits belong to which PR.

I suppose it's a rather special case where we avoid some manual merge conflict resolution at the cost of PR clarity.