Skip to content

Conversation

@Dickson-Mwendia
Copy link
Contributor

Purpose

  • Add code snippet tags for out-of-repo code snippet ingestion

Does this introduce a breaking change?

[ ] Yes [x] No 

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix [ ] Feature [ ] Code style update (formatting, local variables) [ ] Refactoring (no functional changes, no api changes) [X] Documentation content changes [ ] Other... Please describe: 

How to Test

  • Get the code
git clone [repo-address] cd [repo-name] git checkout [branch-name] npm install 
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Dickson-Mwendia Dickson-Mwendia changed the title add snippet tags Add code snippet tags to app.py Jan 24, 2022
Dickson-Mwendia and others added 2 commits January 24, 2022 18:19
Co-authored-by: Chad Kittel <chad.kittel@gmail.com>
@Dickson-Mwendia
Copy link
Contributor Author

Dickson-Mwendia commented Jan 24, 2022

Thanks @ckittel for the call-outs. I've resolved the issues raised as per your feedback. Let me know if it LGTM

Copy link
Member

@ckittel ckittel left a comment

Choose a reason for hiding this comment

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

LGTM!

@ckittel ckittel merged commit a39f41e into Azure-Samples:main Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants