fastdom icon indicating copy to clipboard operation
fastdom copied to clipboard

90 version compat

Open wilsonpage opened this issue 9 years ago • 10 comments

wilsonpage avatar Aug 17 '16 21:08 wilsonpage

Coverage Status

Coverage decreased (-3.6%) to 94.815% when pulling a8145a4402c88632bf0225a5455897ac7c870b10 on 90-version-compat into 5614caa7cb6577810f836bb532098581f1283e42 on master.

coveralls avatar Aug 17 '16 21:08 coveralls

@indianburger can you review this and see if it fixes your issues?

wilsonpage avatar Aug 17 '16 21:08 wilsonpage

Coverage Status

Coverage decreased (-3.6%) to 94.815% when pulling c5caec28f1ec41091c76310a5587e484f75c440c on 90-version-compat into aa3c897bcdff0ee182f29d9a45a03e709f529c8f on master.

coveralls avatar Aug 17 '16 21:08 coveralls

@indianburger I need to write some unit-tests, but don't want to block you 😃

I've added a pretty detailed comment above the new conflict resolution code. It's late and I'm tired, so I could have easily missed something.

wilsonpage avatar Aug 17 '16 21:08 wilsonpage

@indianburger what's your status? Have you found a workaround or are you still interested in this fix?

wilsonpage avatar Aug 22 '16 16:08 wilsonpage

Looks good and works for us. 👍🏾

indianburger avatar Aug 22 '16 16:08 indianburger

Also to note, we hacked a version together to unblock us temporarily. So feel free to take a bit more time to add tests and do it proper.

indianburger avatar Aug 22 '16 16:08 indianburger

Cool. I'm on vacation for the next week. I'll be able to take a look next week.

wilsonpage avatar Aug 22 '16 16:08 wilsonpage

Start

Exkaleburx avatar May 02 '24 05:05 Exkaleburx