Skip to content

Fixed broken links in Markdown Contributing and Readme files#1498

Merged
seantleonard merged 4 commits intoAzure:mainfrom
rodyvansambeek:main
May 18, 2023
Merged

Fixed broken links in Markdown Contributing and Readme files#1498
seantleonard merged 4 commits intoAzure:mainfrom
rodyvansambeek:main

Conversation

@rodyvansambeek
Copy link
Copy Markdown
Contributor

Why make this change?

  • Fixes broken links in Markdown Contributing and Readme files

What is this change?

  • Fix broken links

@rodyvansambeek
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

Copy link
Copy Markdown
Contributor

@severussundar severussundar left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for contributing this change.

Copy link
Copy Markdown
Contributor

@seantleonard seantleonard left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! Links to Learn.Microsoft.com should exclude locale (e.g. `en-us), since the browser will auto-resolve that value.

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread samples/azure/readme.md Outdated
Comment thread samples/azure/readme.md Outdated
@rodyvansambeek
Copy link
Copy Markdown
Contributor Author

Thank you for contributing! Links to Learn.Microsoft.com should exclude locale (e.g. `en-us), since the browser will auto-resolve that value.

Thanks, excuse my oversight. I have changed the URL's without the locale.

Copy link
Copy Markdown
Contributor

@seantleonard seantleonard left a comment

Choose a reason for hiding this comment

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

Thank you for authoring these updates and addressing comments :)

@seantleonard seantleonard enabled auto-merge (squash) May 18, 2023 16:12
@seantleonard
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 6 pipeline(s).

@seantleonard seantleonard merged commit 27bed1f into Azure:main May 18, 2023
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.

3 participants