Skip to content

Tigers - Melley #130

Open
mell2022 wants to merge 3 commits intoAda-C18:mainfrom
mell2022:main
Open

Tigers - Melley #130
mell2022 wants to merge 3 commits intoAda-C18:mainfrom
mell2022:main

Conversation

@mell2022
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@CheezItMan CheezItMan left a comment

Choose a reason for hiding this comment

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

Melley, this is really well written. Very solid code and it all works. Well done, you hit the learning goals here.

Comment thread src/adagrams.js
@@ -1,15 +1,129 @@
const letterPool = Array(9)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 😀

Comment thread src/adagrams.js
Z: 10,
};

export const drawLetters = () => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread src/adagrams.js
return handList;
};

export const usesAvailableLetters = (input, lettersInHand) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 Very clear

Comment thread src/adagrams.js
return true;
};

export const scoreWord = (word) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread src/adagrams.js
return score;
};

export const highestScoreFrom = (words) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

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