search

paulroho / SupermarketKata

A coding kata for implementing a new component in the context of an already existing software.
MIT License
3 stars 3 forks source link

Duplicate ICashRegister #1

Closed dertseha closed 8 years ago

dertseha commented 8 years ago

There is

The recent commit of CashRegister.cs notes a removal of API hints. Which was the intention - removal of ICashRegister from CashRegister.cs or removal of ICashRegister.cs ?

Apart from that: why does this work at all? (I haven't checked whether ICashRegister.cs is in the project.)

paulroho commented 8 years ago

You're right, the ICashRegister.cs should not be there, since the interface is now defined in CashRegister.cs. But note, this is not a problem for the compilation because ICashRegister.cs is not referenced from within the project file.