Skip to content

Add docs regarding BYPASS_TOOL_CONSENT and fix other minor docs issues #82

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

Merged
merged 4 commits into from
Jun 9, 2025

Conversation

mehtarac
Copy link
Member

@mehtarac mehtarac commented Jun 4, 2025

Description

In this PR, I am focusing on three things:

  1. Addition of documentation related to BYPASS_TOOL_CONSENT env variable. This addition is motivated by the issue created here

  2. Fixing diagrams: In the memory_agent.py example, the documentation was still using ASCII diagrams. The change is simply fix this to use Mermaid diagrams to achieve consistency throughout all the docs.

  3. Fix formatting issues for bullet points.

Type of Change

  • New content addition
  • Content update/revision
  • Typo/formatting fix

Motivation and Context

Areas Affected

Screenshots

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using mkdocs serve
  • Links in the documentation are valid and working
  • Images/diagrams are properly sized and formatted
  • All new and existing tests pass

Additional Notes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mehtarac mehtarac requested a review from a team June 4, 2025 17:15
zastrowm
zastrowm previously approved these changes Jun 9, 2025
@mehtarac mehtarac requested a review from zastrowm June 9, 2025 14:42
@mehtarac mehtarac enabled auto-merge (squash) June 9, 2025 15:10
@mehtarac mehtarac merged commit 56bff7b into strands-agents:main Jun 9, 2025
0 of 2 checks passed
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.

2 participants