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

Upgrade eps@1.8.5 #726

Merged
merged 6 commits into from
Jun 26, 2017
Merged

Upgrade eps@1.8.5 #726

merged 6 commits into from
Jun 26, 2017

Conversation

xomaczar
Copy link
Contributor

@xomaczar xomaczar commented Jun 9, 2017

No description provided.

@miguelcobain
Copy link
Owner

@xomaczar we should bump ember-basic-dropdown version as well, to the same version that eps uses, right?

@xomaczar
Copy link
Contributor Author

Forgot about that -- absolutely.

@miguelcobain
Copy link
Owner

I'm concerned if this upgrade is just a matter of bumping versions. Please test if this upgrade breaks any functionality.

@xomaczar
Copy link
Contributor Author

You don't trust our test suite?

@miguelcobain
Copy link
Owner

@xomaczar unfortunately some things are still poorly tested, e.g paper-select, only two tests there

@miguelcobain
Copy link
Owner

But overall, I think we have very good tests.

@xomaczar
Copy link
Contributor Author

xomaczar commented Jun 22, 2017 via email

@bjornharrtell
Copy link
Contributor

bjornharrtell commented Jun 24, 2017

Took a look at this as I have personal interest in this upgrade and looked into upgrading but @xomaczar got there first with what looks like clean changes and nice removals of no longer needed code. Tried out the branch locally and tried the autocomplete, menu and select components without any issues. Great work @xomaczar! :)

@miguelcobain
Copy link
Owner

Great PR. 👌
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants