Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

added readme for algorithms sudoku solver #556

Merged
merged 6 commits into from
Oct 4, 2021

Conversation

ycechungAI
Copy link
Contributor

added readme for sudoku solver under algoirthms

need more instructions to add machine learning README

@ghost
Copy link

ghost commented Sep 27, 2021

CLA assistant check
All CLA requirements met.

@ycechungAI
Copy link
Contributor Author

#432 added

Copy link
Member

@msoeken msoeken left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @ycechungAI, looks good to me after a few suggestions.

samples/machine-learning/README.md Outdated Show resolved Hide resolved
samples/machine-learning/README.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@ycechungAI ycechungAI left a comment

Choose a reason for hiding this comment

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

not sure about these warnings for unknown word netcoreapp, but shouldnt affec't the readme.

@msoeken
Copy link
Member

msoeken commented Oct 4, 2021

One CI error was due to multiple empty lines in Markdown. I fixed this problem in the last commit. Let's await CI.

@msoeken msoeken added the Hacktoberfest Good issue for Hacktoberfest 2021 participants! label Oct 4, 2021
@msoeken msoeken merged commit 5c96c6c into microsoft:main Oct 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Hacktoberfest Good issue for Hacktoberfest 2021 participants!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants