Skip to content

Remove the dead code show_agent_logs #3229

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

beinan
Copy link

@beinan beinan commented Jul 27, 2025

Looks like the execution path of the agents are going to handle_agent_logs_execution in the Console Formatter rather than the code removed.

@lucasgomide
Copy link
Contributor

lucasgomide commented Aug 6, 2025

@beinan would you mind to sync your branch?

edit: also fix linter issues pls

@beinan beinan force-pushed the beinan/remove_dead_code branch from b741eee to b369f47 Compare August 8, 2025 04:04
@beinan beinan force-pushed the beinan/remove_dead_code branch from b369f47 to ef1df0d Compare August 8, 2025 04:15
@beinan
Copy link
Author

beinan commented Aug 8, 2025

@beinan would you mind to sync your branch?

edit: also fix linter issues pls

Rebased and fixed linter issues
Thank you @lucasgomide for looking into this pr!

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