8000 Update ember-basic-dropdown to latest release by andrewpye · Pull Request #681 · smile-io/ember-polaris · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 2, 2024. It is now read-only.

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 requested review from vladucu and tomnez November 18, 2020 10:07
@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 andrewpye requested a review from vladucu November 18, 2020 10:38
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0