Skip to content
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

Move agentops.enums to their respective module #647

Merged
merged 1 commit into from
Jan 16, 2025
Merged

Conversation

teocns
Copy link
Contributor

@teocns teocns commented Jan 16, 2025

Removing a redundant module agentops.enums and instead moving them in the modules they belong:

  • Moved EventType enum from enums.py to event.py
  • Moved EndState enum from enums.py to session.py
  • Removed enums.py module
  • Updated imports across affected files
  • Updated dependency graph in tach.yml

@teocns teocns changed the title Move Enums to Their Respective Modules Move agentops.enums to their respective module Jan 16, 2025
Copy link

codecov bot commented Jan 16, 2025

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
agentops/partners/autogen_logger.py 0.00% 1 Missing ⚠️
agentops/session.py 85.71% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@teocns teocns merged commit 6d0459a into main Jan 16, 2025
10 checks passed
@teocns teocns deleted the refactor-enums branch January 16, 2025 21:31
@teocns teocns added the v0.4 label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant