-
Notifications
You must be signed in to change notification settings - Fork 607
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 dependency @ember/string to v4 #9011
Conversation
☔ The latest upstream changes (presumably 3e3c470) made this pull request unmergeable. Please resolve the merge conflicts. |
77083bd
to
ac7b272
Compare
☔ The latest upstream changes (presumably a8f3ae0) made this pull request unmergeable. Please resolve the merge conflicts. |
ac7b272
to
ebd12c9
Compare
☔ The latest upstream changes (presumably 9188850) made this pull request unmergeable. Please resolve the merge conflicts. |
ebd12c9
to
9db7627
Compare
☔ The latest upstream changes (presumably 67c0877) made this pull request unmergeable. Please resolve the merge conflicts. |
a40c9ba
to
069ba37
Compare
☔ The latest upstream changes (presumably 81f3cf9) made this pull request unmergeable. Please resolve the merge conflicts. |
c3780bd
to
94cdbd0
Compare
☔ The latest upstream changes (presumably fa0a007) made this pull request unmergeable. Please resolve the merge conflicts. |
94cdbd0
to
dc318fd
Compare
☔ The latest upstream changes (presumably 801cc33) made this pull request unmergeable. Please resolve the merge conflicts. |
067f115
to
c5a30b4
Compare
☔ The latest upstream changes (presumably 1c8c8ff) made this pull request unmergeable. Please resolve the merge conflicts. |
c5a30b4
to
a370929
Compare
a370929
to
206bd8e
Compare
... instead of relying on the package itself which is causing build issues again and again
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
☔ The latest upstream changes (presumably 4961e64) made this pull request unmergeable. Please resolve the merge conflicts. |
@@ -0,0 +1,9 @@ | |||
// Imported from the `@ember/string` package to avoid build issues... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean, if this is all you need, then it's probably more valuable to try to remove the dep
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's all we need in the app itself, but it looks like e.g. ember-tooltips needs more
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like ember-tooltips could also do with a v2 conversion 🤔
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
3.1.1
->4.0.0
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.