Make order of cookie loading with the same name configurable #248

merged 6 commits into from Sep 18, 2012


None yet

3 participants

YUI Library member

This is a merged version of Pull Request #196.

I pulled it in, merged the difference changes from 3.x and ran the tests. They all pass and it's still at 97% and 100% coverage.

I recommend merging into 3.x for the next release.

YUI Library member

Hey @theaxel, it would help if you had a CLA signed with us.

I don't personally think one is needed for this, as it's a pretty simple and small change.


Hi @davglass

i already signed the CLA a couple of month ago. And thanks for the merge.

YUI Library member

Our CLA checker didn't see that it was approved, I pinged to get it checked and they tell me it's ok. Sorry about looking like I didn't know WTF I was talking about :)

Anyway, I'm trying to get this into the next release. Sorry for the delay, we don't have a core owner for Cookie, so I just volunteered to handle it.

YUI Library member

👍 Merging in shortly

@yuibuild yuibuild merged commit bb6652d into yui:3.x Sep 18, 2012

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment