Skip to content

RhysFJ RPS Progress#228

Open
RhysFJohn wants to merge 2 commits intomakersacademy:mainfrom
RhysFJohn:main
Open

RhysFJ RPS Progress#228
RhysFJohn wants to merge 2 commits intomakersacademy:mainfrom
RhysFJohn:main

Conversation

@RhysFJohn
Copy link

No description provided.

@@ -0,0 +1,5 @@
<div class="results">
<h2> <%= @game.player %>, you chose <%= @game.player_select %> </h2>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opinion: from what I understand, our erb files should be light in code, so it would be good to call the method on @GAMe outside of this file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants