Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The group comboBase should inherit from the default if no group setting is specified #1837

Closed
wants to merge 1 commit into from

Conversation

andrewnicols
Copy link
Contributor

This fixes the regression caused by #1832.

The other options (combine, comboSep, maxURLLength) are also affected in a similar way, but these are not a regression introduced in 3.17 so I'll submit those in a separate PR.

@yahoocla
Copy link

CLA is valid!

@andrewnicols
Copy link
Contributor Author

Darn - that should have been dev-master not dev-3.x in the PR.

@juandopazo
Copy link
Member

@andrewnicols thanks! I guess we'll need 3.17.2.

@@ -1,10 +1,10 @@
YUI Loader Change History
=========================

@VERSION@
3.17.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The release process handles replacing @VERSION@. You should leave it.

…ng is specified

This fixes a regression caused by yui#1832.
@andrewnicols
Copy link
Contributor Author

D'oh! I managed to not change it on another commit I did today. Brain everyhwere. New commit awaits your approval.

Thanks,

Andrew

@F21
Copy link

F21 commented May 23, 2014

Thanks for fixing this! I just noticed it after upgrading to 3.17.1.

@juandopazo
Copy link
Member

@andrewnicols I guess all of our brains where all over the place this week.

@triptych
Copy link
Contributor

@andrewnicols
Copy link
Contributor Author

I guess it's too late for that spelling fix - oops.

@tripp
Copy link
Contributor

tripp commented May 23, 2014

@andrewnicols
I can correct the typo and merge it into the branch so it will land in the 3.17.2 release.

@tripp
Copy link
Contributor

tripp commented May 23, 2014

#1840

@unkillbob
Copy link
Contributor

FYI I have verified the 3.17.2 RC in our environment and am happy that the issue is resolved for us :)

@tripp
Copy link
Contributor

tripp commented Aug 15, 2014

This was merged into 3.17.2

@tripp tripp closed this Aug 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants