Ax icon indicating copy to clipboard operation
Ax copied to clipboard

Remove display_cards

Open mpolson64 opened this issue 8 months ago • 3 comments

Summary: Now that list[AnalysisCard] displays all the cards there is no need for this method.

Differential Revision: D71912435

mpolson64 avatar Mar 26 '25 18:03 mpolson64

This pull request was exported from Phabricator. Differential Revision: D71912435

facebook-github-bot avatar Mar 26 '25 18:03 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D71912435

facebook-github-bot avatar Mar 26 '25 19:03 facebook-github-bot

Codecov Report

Attention: Patch coverage is 50.00000% with 12 lines in your changes missing coverage. Please review.

Project coverage is 95.98%. Comparing base (46a77b3) to head (6789556).

Files with missing lines Patch % Lines
ax/analysis/analysis.py 42.85% 12 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3569      +/-   ##
==========================================
- Coverage   96.01%   95.98%   -0.03%     
==========================================
  Files         540      540              
  Lines       53230    53240      +10     
==========================================
- Hits        51108    51104       -4     
- Misses       2122     2136      +14     

: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.

codecov-commenter avatar Mar 26 '25 19:03 codecov-commenter