Repository navigation
Add lstk to all the README pages where the reference was not listed in the prerequisites - #137
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Duplicate prerequisites, broken documentation links, and the unstated SQL EULA scope should be addressed.
12 open findings
What changed in this PR
Adds missing lstk CLI prerequisites across Azure samples and introduces SQL Server EULA setup guidance.
Changes:
- Adds
lstk CLIto 21 prerequisite sections. - Adds SQL Server EULA/configuration guidance to eight SQL sample guides.
- Contains duplicate prerequisites and broken root-relative documentation links.
| File | Description |
|---|---|
samples/web-app-sql-database/python/terraform/README.md |
Adds SQL EULA guidance. |
samples/web-app-sql-database/python/scripts/README.md |
Adds SQL EULA guidance. |
samples/web-app-sql-database/python/README.md |
Adds prerequisite and EULA guidance. |
samples/web-app-sql-database/python/bicep/README.md |
Adds SQL EULA guidance. |
samples/web-app-sql-database/dotnet/terraform/README.md |
Adds SQL EULA guidance. |
samples/web-app-sql-database/dotnet/scripts/README.md |
Adds SQL EULA guidance. |
samples/web-app-sql-database/dotnet/README.md |
Adds prerequisite and EULA guidance. |
samples/web-app-sql-database/dotnet/bicep/README.md |
Adds SQL EULA guidance. |
samples/web-app-postgresql-flexible-server/python/README.md |
Adds lstk prerequisite. |
samples/web-app-postgresql-flexible-server/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-mysql-flexible-server/python/README.md |
Adds lstk prerequisite. |
samples/web-app-mysql-flexible-server/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-managed-identity/python/README.md |
Adds lstk prerequisite. |
samples/web-app-managed-identity/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-custom-image/python/README.md |
Adds lstk prerequisite. |
samples/web-app-custom-image/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-cosmosdb-nosql-api/python/README.md |
Adds lstk prerequisite. |
samples/web-app-cosmosdb-nosql-api/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-cosmosdb-mongodb-api/python/README.md |
Adds lstk prerequisite. |
samples/web-app-cosmosdb-mongodb-api/dotnet/README.md |
Adds lstk prerequisite. |
samples/web-app-app-configuration/python/README.md |
Adds lstk prerequisite. |
samples/web-app-app-configuration/dotnet/README.md |
Adds lstk prerequisite. |
samples/servicebus/java/README.md |
Adds a duplicate lstk prerequisite. |
samples/function-app-storage-http/dotnet/README.md |
Adds an lstk prerequisite. |
samples/function-app-service-bus/dotnet/README.md |
Adds lstk prerequisite. |
samples/function-app-managed-identity/python/README.md |
Adds lstk prerequisite. |
samples/function-app-front-door/python/README.md |
Adds a duplicate lstk prerequisite. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Removed duplicate lstk CLI entry from prerequisites. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated the lstk CLI entry for clarity. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Removed duplicate entry for lstk CLI in prerequisites. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README.md to use HTTPS format and clarified environment variable settings for the emulator. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README to use HTTPS and fixed formatting issues. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README to use HTTPS and fixed formatting. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
DrisDary
approved these changes
Oct 14, 2026
Updated links in README to use proper formatting and added note about setting MSSQL_ACCEPT_EULA in the container environment. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README to use proper formatting and fixed EULA note. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README to use proper formatting. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README.md to use HTTPS format. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Updated links in README to use HTTPS and fixed formatting. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
This pull requests adds lstk CLI reference in the prerequisites section in all the
READMEfiles where the link was missing.Fixes SMF-945