Skip to content

Conversation

@crenshaw-dev
Copy link
Contributor

I noticed the need for this enhancement while fixing: #964

The status field validation error was reported only in logs. If we instead stored it in the ready condition on the status, the failure would be available in the Argo CD UI.

@codecov-commenter
Copy link

Bundle Report

Bundle size has no change ✅

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.83%. Comparing base (106f026) to head (e2a6eae).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #965      +/-   ##
==========================================
+ Coverage   50.69%   50.83%   +0.14%     
==========================================
  Files          51       51              
  Lines        5494     5526      +32     
==========================================
+ Hits         2785     2809      +24     
- Misses       2408     2411       +3     
- Partials      301      306       +5     

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

@crenshaw-dev crenshaw-dev marked this pull request as ready for review January 29, 2026 19:28
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.

2 participants