[MagenticOne] Report the error reason in the exception and skip the s…
…econd check if an error exists
Why are these changes needed?
Related issue number
Checks
- [ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://github.com/microsoft/autogen/blob/main/CONTRIBUTING.md to build and test documentation locally.
- [ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
- [ ] I've made sure all auto checks have passed.
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 81.28%. Comparing base (13e144e) to head (4b3b95c).
Additional details and impacted files
@@ Coverage Diff @@
## main #7039 +/- ##
==========================================
+ Coverage 81.22% 81.28% +0.05%
==========================================
Files 244 244
Lines 18512 18517 +5
==========================================
+ Hits 15036 15051 +15
+ Misses 3476 3466 -10
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 81.28% <100.00%> (+0.05%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@smalltalkman thanks, could you fix the build errors? You can instruct your coding agent to follow instructions in https://github.com/microsoft/autogen/blob/main/.github/copilot-instructions.md
@ekzhu Thanks for your guidance. The type check error has been fixed.
@microsoft-github-policy-service agree