feat(bar_chart): color overrides

[nickofthyme]

Summary

Allow user to override colors of given datum based on accessor function.

interface RawDataSeriesDatum {
  x: number | string;
  y1: number | null;
  y0?: number | null;
  datum?: any;
}
interface GeometryId {
  specId: SpecId;
  seriesKey: any[];
}
type Color = string;
type StyleOverride = RecursivePartial<BarSeriesStyle> | Color | null;
type StyleAccessor = (datum: RawDataSeriesDatum, geometryId: GeometryId) => StyleOverride;

BREAKING CHANGE: Replace ColorAccessor prop and functionality with StyleAccessor

#216

Checklist

• Proper documentation or storybook story was added for features that require explanation or tutorials
• Unit tests were updated or added to match the most common scenarios
• Each commit follows the convention

[nickofthyme]

I didn't label these changes as breaking because I don't think anyone is using the colorAccessors attribute of the SeriesSpec. But technically, I am removing colorAccessors in place of colorAccessor.

[codecov-io]

Codecov Report

Merging #271 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
+ Coverage   98.05%   98.05%   +<.01%     
==========================================
  Files          37       37              
  Lines        2671     2680       +9     
  Branches      613      615       +2     
==========================================
+ Hits         2619     2628       +9     
  Misses         47       47              
  Partials        5        5

Impacted Files	Coverage Δ
src/chart_types/xy_chart/utils/specs.ts	100% <ø> (ø)	⬆️
src/chart_types/xy_chart/store/utils.ts	96.68% <ø> (ø)	⬆️
src/chart_types/xy_chart/domains/y_domain.ts	100% <ø> (ø)	⬆️
src/chart_types/xy_chart/rendering/rendering.ts	99.32% <100%> (+0.04%)	⬆️
src/chart_types/xy_chart/utils/series.ts	100% <100%> (ø)	⬆️

---

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 c8b206c...c87a7bb. Read the comment docs.

[markov00]

Not already tested locally, but I've left few comments.
I think it's better to specify that as a breaking change to correctly align the semver with the code changes also if no one is using the colorAccessor

[nickofthyme]

@markov00 Could you review this again when you have time?

[markov00]

Code LGTM!
Remember to add the BREAKING CHANGE footer on the merge commit with the description of the changes.
Thanks!

[markov00]

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀