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

Update ember-basic-dropdown to latest release #681

Merged
merged 4 commits into from
Nov 18, 2020

Conversation

andrewpye
Copy link
Member

@andrewpye andrewpye commented Nov 18, 2020

Updates ember-basic-dropdown to the most recent release (v3.0.11) with the aim of fixing a build error being seen in other ember-polaris PRs. The breaking changes involved in this update can be found in the ember-basic-dropdown release notes.

@andrewpye andrewpye self-assigned this Nov 18, 2020
Copy link
Member

@vladucu vladucu left a comment

Choose a reason for hiding this comment

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

This looks neat and simple!
But, latest version seems to be 3.x, can we get on that one? See changelog

@andrewpye
Copy link
Member Author

Oof. I got mixed up after seeing this while scrolling through releases:

image

and thought that it referred to v2.0.15 🙄

@vladucu
Copy link
Member

vladucu commented Nov 18, 2020

yeah, been there done that 😂

@andrewpye
Copy link
Member Author

Welp, looks like v2.0.15 doesn't fix the build error on Canary anyway. Here's hoping v3 does... 🙏

@andrewpye
Copy link
Member Author

Turns out even latest ember-basic-dropdown still has a build issue on Canary at the moment 🤷‍♂️ We're going to allow this for now and update when a fix is available for ember-basic-dropdown.

Copy link
Member

@vladucu vladucu left a comment

Choose a reason for hiding this comment

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

@andrewpye before merging, let's tweak this to allow canary build to fail by configuring it in config/ember-try.js (check allowedToFail in ember-try)

@andrewpye andrewpye added the dependencies Pull requests that update a dependency file label Nov 18, 2020
@andrewpye andrewpye merged commit 8ea2fb6 into master Nov 18, 2020
@andrewpye andrewpye deleted the update/ember-basic-dropdown branch November 18, 2020 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants