Skip to content

Conversation

@acroca
Copy link
Member

@acroca acroca commented Jan 14, 2026

We've recently added new ext packages but didn't add them to the ci triggers.

Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca acroca requested review from a team as code owners January 14, 2026 11:00
@codecov
Copy link

codecov bot commented Jan 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.96%. Comparing base (bffb749) to head (da5daec).
⚠️ Report is 62 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #886      +/-   ##
==========================================
+ Coverage   86.63%   86.96%   +0.33%     
==========================================
  Files          84      103      +19     
  Lines        4473     6898    +2425     
==========================================
+ Hits         3875     5999    +2124     
- Misses        598      899     +301     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@CasperGN CasperGN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - @acroca I'll put it on my mental backlog to come back and make the trigger for tests for strands/langgraph to be dependent on changes in the folder structure. @sicoyle and I are also working on having more stable llm runs in ci and will come back with these updates once ready.

@acroca acroca added this pull request to the merge queue Jan 16, 2026
Merged via the queue into dapr:main with commit 9f89746 Jan 16, 2026
12 of 13 checks passed
@acroca acroca deleted the add-missing-ci-triggers branch January 16, 2026 08:49
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.

3 participants