param icon indicating copy to clipboard operation
param copied to clipboard

Validate ParamOverrides

Open maximlt opened this issue 1 year ago • 1 comments

I just chatted with @hoxbro and told him I had this in my stash. Totally untested at this stage. Wildly simpler than the alternative implementation attempted in https://github.com/holoviz/param/pull/92, which I think focused quite a bit on getting things perfectly right with Dynamic Parameters.

  • [ ] Add tests
  • [ ] Run HoloViews' test suite with Param's code checkout from this PR
  • [ ] Warn first when the validation fails, letting users know in the future this will raise an error

Fixes https://github.com/holoviz/param/issues/85

maximlt avatar Mar 07 '25 13:03 maximlt

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.46%. Comparing base (90303ac) to head (1da4ee7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1041   +/-   ##
=======================================
  Coverage   87.45%   87.46%           
=======================================
  Files           9        9           
  Lines        4959     4962    +3     
=======================================
+ Hits         4337     4340    +3     
  Misses        622      622           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: 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.

codecov[bot] avatar Mar 07 '25 13:03 codecov[bot]