zackads / mumble

https://learn.madetech.com/katas/mumbling/
0 stars 0 forks source link

Feature complete: Mumbles two capital letters #2

Closed zackads closed 4 years ago

zackads commented 4 years ago

Context

I believe we have triangulated the tested behaviour of mumbling two capital letters sufficiently to move on to the next feature.

Changes proposed in this pull request

This PR adds the ability to mumble two capital letters through concatenation of input substrings around a hyphen.

Guidance to review

I have used the constant -> scalar transformation to deliver this feature. Though it is not scalable for future possible features, I believe it is the smallest and most simple production code that will pass tests.

I have a feeling that future tests may move us away from this approach...

Link to issue

N/A - No issue raised (I will raise an issue for our next feature 🤓)