-
Notifications
You must be signed in to change notification settings - Fork 771
Removing !developer_secrets from all docs #496
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
Changes from all commits
a2fd842
90f7aef
5de24f7
23993ca
ae79927
8ae3d91
e0d3dfa
62e9b10
59b9414
6b925a2
00c1bd8
d880ab4
968a38f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,7 +38,7 @@ The SDK uses a streamlined approach to secrets management: | |
|
||
Two types of secrets are supported: | ||
|
||
1. **Developer Secrets** (`!developer_secret`): | ||
1. **Developer Secrets**: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I didn't see your PR before I did the same. Your's is good, but we should also make some other changes in this README (e.g. see main...fix/update-docs-for-secrets-tags) and should delete There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated here -- #497 |
||
|
||
- Used for secrets that are provided by developers | ||
- Values are known at deployment time | ||
|
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.
🛠️ Refactor suggestion
🧩 Analysis chain
Add a quick self-check to keep MD040 clean going forward.
Run markdownlint locally to verify no remaining MD040 in this file.
🏁 Script executed:
Length of output: 6704
Add markdownlint self-check and fix MD040 occurrences
markdownlint-cli2 found 50 errors in examples/mcp_agent_server/asyncio/README.md, including multiple MD040 (fenced code blocks missing language) — e.g. lines 102, 108, 121, 139, 159, 174, 181, 256, 293, 333. Action items:
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
159-159: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
167-167: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
174-174: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
181-181: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents