Skip to content
This repository was archived by the owner on Aug 21, 2024. It is now read-only.

Comments

Workbook for TruthTables#673

Merged
tcNickolas merged 3 commits intomicrosoft:mainfrom
olstulo:olstulo/Workbook4TruthTables
Nov 30, 2021
Merged

Workbook for TruthTables#673
tcNickolas merged 3 commits intomicrosoft:mainfrom
olstulo:olstulo/Workbook4TruthTables

Conversation

@olstulo
Copy link
Contributor

@olstulo olstulo commented Oct 15, 2021

Added a 1st draft of a workbook for TruthTables

@ghost
Copy link

ghost commented Oct 15, 2021

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@tcNickolas tcNickolas left a comment

Choose a reason for hiding this comment

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

Looks good! I went ahead and added the per-task links between the kata and the workbook, and the code cells in the workbook that show the complete solutions. I also wrote up a bit more details for the tasks that needed them - most katas don't use user-defined types, so I imagine the syntax for dealing with them will be new to a lot of users. I also removed the last two tasks which don't have solutions - not having a solution explained in the workbook is enough, we don't need to put additional pressure on ourselves by saying "coming soon" :-)

Thank you! Let me know if you'd like to continue working on the solutions to the last two tasks!

@tcNickolas tcNickolas merged commit 28fcb70 into microsoft:main Nov 30, 2021
@olstulo
Copy link
Contributor Author

olstulo commented Nov 30, 2021

Looks good! I went ahead and added the per-task links between the kata and the workbook, and the code cells in the workbook that show the complete solutions. I also wrote up a bit more details for the tasks that needed them - most katas don't use user-defined types, so I imagine the syntax for dealing with them will be new to a lot of users. I also removed the last two tasks which don't have solutions - not having a solution explained in the workbook is enough, we don't need to put additional pressure on ourselves by saying "coming soon" :-)

Thank you! Let me know if you'd like to continue working on the solutions to the last two tasks!

Thank you for taking the time to review my PR and making it merge ready! And yes, I'd like to continue working on the last 2 solutions.

@tcNickolas
Copy link
Contributor

Great, thank you!!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants