Skip to content

Conversation

@caroljung-cg
Copy link
Contributor

@caroljung-cg caroljung-cg commented Feb 12, 2025

Changes

  • Decouples auth from codegen session
  • Create a global session manager for headless compute

Content

Testing

Please check the following before marking your PR as ready for review

  • I have added tests for my changes
  • I have updated the documentation or added new documentation as needed

@codecov
Copy link

codecov bot commented Feb 12, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
30 2 28 0
View the top 2 failed test(s) by shortest run time
::tests.unit.codegen.sdk.core.test_directory
Stack Traces | 0s run time
ImportError while importing test module '.../sdk/core/test_directory.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
.../sdk/core/test_directory.py:8: in <module>
    from codegen.sdk.codebase.codebase_graph import CodebaseGraph
E   ModuleNotFoundError: No module named 'codegen.sdk.codebase.codebase_graph'
::tests.unit.codegen.shared.configs.test_constants
Stack Traces | 0s run time
ImportError while importing test module '.../shared/configs/test_constants.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
.../shared/configs/test_constants.py:3: in <module>
    from codegen.shared.configs.constants import (
E   ImportError: cannot import name 'ENV_FILENAME' from 'codegen.shared.configs.constants' (.../shared/configs/constants.py)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@caroljung-cg caroljung-cg changed the title CG-10779: Decouple auth from codegen session feat: CG-10779: Global session management + decoupled auth Feb 12, 2025
@caroljung-cg caroljung-cg marked this pull request as ready for review February 12, 2025 20:08
@caroljung-cg caroljung-cg requested review from a team and codegen-team as code owners February 12, 2025 20:08
@caroljung-cg caroljung-cg merged commit eb1647b into develop Feb 12, 2025
25 of 26 checks passed
@caroljung-cg caroljung-cg deleted the cg-10779-separate-session-auth branch February 12, 2025 20:10
@github-actions
Copy link
Contributor

🎉 This PR is included in version 0.11.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants