Isoldhe / Battleship

Battleship game in C#
0 stars 1 forks source link

Add Display and DisplayElement classes #11

Closed Atrixsanna closed 5 years ago

Atrixsanna commented 5 years ago

step in the direction of solving issue #4

Atrixsanna commented 5 years ago

closes #8 because the new BoardPosition now maps the battlefield coordinates to an object in a dictionary

Atrixsanna commented 5 years ago

closes #4 because the functionality has been added in this pull request

Isoldhe commented 5 years ago

You're going so fast 😅 Leave some for me 😆 I'll probably be able to do something this weekend.

Atrixsanna commented 5 years ago

You're going so fast 😅 Leave some for me 😆 I'll probably be able to do something this weekend.

Hehehe, don't worry, plenty issues to solve and more to come. Every feature is a new challenge 😄 I'm mostly trying to lay a good concrete slab of groundwork here and part of all this is actually also still figuring out how to properly handle console input and output - I'm learning too 😉

Do you wish to further review this pull request or can it be merged?

Isoldhe commented 5 years ago

I think it looks good 👍 you can merge it.