search

joshuaulrich / xts

Extensible time series class that provides uniform handling of many R time series classes by extending zoo.
http://joshuaulrich.github.io/xts/
GNU General Public License v2.0
219 stars 70 forks source link

Merge suffixes handling #372

Closed pierre-lamarche closed 1 year ago

pierre-lamarche commented 1 year ago

Related to issue #371. I am encountering troubles when running unit tests, seemingly not related to the changes in the merge function, as it is also the case when I run unit tests on the master branch. More likely related to my config, I am digging to find the reason for that.

However the changes work fine and I get the expected behavior.

joshuaulrich commented 1 year ago

This looked good to me. Thanks for the report and PR!