-
Notifications
You must be signed in to change notification settings - Fork 356
fix: Dev v3 to main #549
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
NirajC-Microsoft
wants to merge
91
commits into
main
Choose a base branch
from
dev-v3
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
fix: Dev v3 to main #549
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removed mcp dependency and updated formatting.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Refactored imports and added a successful check helper function for health checks.
Removed duplicate abstract method definition for get_agent_messages.
fix: Pylint errors fixes
fix: ignore test_otlp_tracing.py file in test.yml
…eady existed agents are not using the previous resource group connection
fix: added the lifespan of the agent on container stop and checked the already existed agents are not using the previous resource group connection
fix: Unavailable quota handling in quotacheck.sh file
refactor: removes the use of `contextvars` and refactors the codebase to explicitly pass `user_id`
…ipts for users deployed using AVM template (#550) * updated post deployment scripts for avm deployment * fix update function call typo * refactor: improve extraction of deployment outputs in team_config_and_data.sh * add AVM Post Deployment Guide * update AVM Post Deployment Guide
feat: UI updates for the Alert Message pop-up when a user attempts to navigate to another page while a plan is in progress
fix: Disable 3 dots option from Default Teams(fix for bug 23528)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request primarily refactors import paths throughout the test suite to use explicit, absolute imports from the
src.backend
andsrc.mcp_server
packages. This change improves code clarity, maintainability, and consistency as the project structure evolves. Additionally, the test workflow configuration is updated to ignore more test files during coverage runs, streamlining CI behavior.Test workflow improvements
.github/workflows/test.yml
workflow to ignore additional test files during coverage runs, reducing unnecessary coverage reporting and focusing on relevant tests.Refactoring import paths in backend tests
test_auth_utils.py
,test_sample_user.py
,test_health_check.py
,test_messages.py
,test_config.py
, andtest_otlp_tracing.py
) to use explicit paths from thesrc.backend
package, improving clarity and consistency. [1] [2] [3] [4] [5] [6]Refactoring import paths in mcp_server and agent tests
test_foundry_integration.py
,test_factory.py
,test_hr_service.py
, andtest_utils.py
) to use absolute paths fromsrc.backend.v3.magentic_agents
andsrc.mcp_server
, ensuring proper module resolution. [1] [2] [3] [4]Minor code cleanup
AgentMessageData
model inmessages_kernel.py
.Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information