-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add readme check #2955
Add readme check #2955
Conversation
…xamples into add-readme-check
…xamples into add-readme-check
…automl-standalone-jobs/cli-automl-forecasting-task-github-dau
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Docs approval. These changes will not break our build.
I re-opened the previous PR but addressed the notebooks that were failing. I figured out the issue, so we can discuss in our meeting. Thanks for reviewing! |
I've confirmed these changes don't break the azure-docs-pr build using learn-validation in VSCode. |
Description
This PR adds a step to the CI which will check that each sample folder has a README file following the README template to ensure that all samples are descriptive enough for users to easily understand their purpose and if they're relevant. To start, all sample directories without README files will be placed into an excluded list and bypassed, but all owners of such samples will be contacted and asked to provide READMEs.
Checklist