Ole

Results 7 comments of Ole

@MounirDhahri is this ready to be merged?

It seems like `copper` has been [removed](https://github.com/artsy/palette/commit/453b1c351743d1f1df690440138fe16778cc2a85) from Palette but it's still [used](https://app.circleci.com/pipelines/github/artsy/eigen/28673/workflows/acb3f463-5f2f-4cf7-9994-81e05a5aa558/jobs/85495) in Eigen.

> @olerichter00 do you think we should keep this open or should we close for now? @gkartalis we can close this for now and reopen in case I pick the...

Looks like this PR is ready to be merged 🚢

> @olerichter00 addressed all your comments - let me know if I need to make any other changes That was fast! I've looked through the code again, and everything looks...

> Ideally, we would run tests only in files that are importing that method but that might not be trivial. What do you think about it? Yes, that would be...

> @olerichter00 I am not sure what you mean by moving the data to MP - I assumed it's already there now Sorry, this is a bit ambiguous. I was...