Rename "details" to "metadata" for data check actions

[angela97lin]

Closes #1988

[codecov]

Codecov Report

Merging #2008 (e9dc772) into main (cdb7699) will not change coverage.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##             main    #2008   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         278      278           
  Lines       22748    22748           
=======================================
  Hits        22739    22739           
  Misses          9        9           

Impacted Files	Coverage Δ
evalml/data_checks/highly_null_data_check.py	100.0% <ø> (ø)
evalml/data_checks/id_columns_data_check.py	100.0% <ø> (ø)
evalml/data_checks/no_variance_data_check.py	100.0% <ø> (ø)
evalml/data_checks/sparsity_data_check.py	100.0% <ø> (ø)
evalml/data_checks/target_leakage_data_check.py	100.0% <ø> (ø)
evalml/data_checks/uniqueness_data_check.py	100.0% <ø> (ø)
...s/data_checks_tests/test_highly_null_data_check.py	100.0% <ø> (ø)
...ts/data_checks_tests/test_id_columns_data_check.py	100.0% <ø> (ø)
...ests/data_checks_tests/test_sparsity_data_check.py	100.0% <ø> (ø)
...ata_checks_tests/test_target_leakage_data_check.py	100.0% <ø> (ø)
... and 6 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdb7699...e9dc772. Read the comment docs.

[freddyaboulton]

Looks great to me! Is there a similar plan for the warning/error message classes?

[angela97lin]

@freddyaboulton Currently, no--I think this came up for actions because they're used as data/information elsewhere, whereas for "messages", details seems okay 😅 Open to discussion though, I could see it being a good idea for consistency, and also a good idea to keep as "details" to differentiate? :)

[jeremyliweishih]

Looks great! Nice refactor

[bchen1116]

The attention to detail metadata