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 dependencies #4253

Merged
merged 3 commits into from
Dec 13, 2021
Merged

Conversation

mydea
Copy link
Member

@mydea mydea commented Dec 9, 2021

This PR updates dependencies for @sentry/ember to their latest version:

  • ember-cli-htmlbars from 5.7.1 to 6.0.1
    • The breaking change here is dropping older node versions that are not supported here anyhow
  • ember-cli-typescript from 3.1.4 to 4.2.1
    • The breaking change here is also for dropping older node versions, and custom babel config which is not used here as far as I see
    • I also applied the changes that have been changed in the ember-cli-typescript default blueprint
  • ember-auto-import allow both 1.x and 2.x
    • No custom babel config is used, so both should work just fine
    • Just needed to add webpack as a dev dependency, as that is needed by auto-import v2

@AbhiPrasad AbhiPrasad requested a review from k-fish December 9, 2021 15:04
@@ -1,7 +1,7 @@
{
"extends": "../../tsconfig.json",
"compilerOptions": {
"target": "es2017",
"target": "es2020",
Copy link
Member

Choose a reason for hiding this comment

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

Why did we bump this?

Copy link
Member Author

Choose a reason for hiding this comment

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

It was bumped in the ember-cli-typescript blueprint from 3.x to 4.x. I can also leave it as it was?

Copy link
Member

Choose a reason for hiding this comment

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

I would prefer not to bump the target if we don't need to.

Copy link
Member Author

Choose a reason for hiding this comment

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

sure thing, I reverted it!

@mydea mydea force-pushed the fn/ember-auto-import branch from b7c0c39 to 7a6c173 Compare December 9, 2021 15:24
@AbhiPrasad
Copy link
Member

AbhiPrasad commented Dec 9, 2021

Will this make @sentry/ember Ember 4.x compatible?

@mydea
Copy link
Member Author

mydea commented Dec 10, 2021

I have not tried it yet, but it should work just fine I believe, if ember-auto-import v1 is the blocker :) I'll try to give it a spin in an ember 4 app!

@Emerson
Copy link

Emerson commented Dec 10, 2021

Very excited for this change 😁

@AbhiPrasad
Copy link
Member

@mydea, mind rebasing the branch? We can ship this with the next release!

@xomaczar
Copy link

Same for us - currently blocked upgrading to ember4.0

@mydea mydea force-pushed the fn/ember-auto-import branch from 7a6c173 to 6bf2ef0 Compare December 13, 2021 14:36
@mydea
Copy link
Member Author

mydea commented Dec 13, 2021

I rebased and pushed, tests are running!

@AbhiPrasad AbhiPrasad merged commit 582d678 into getsentry:master Dec 13, 2021
@AbhiPrasad
Copy link
Member

Merged! Next release will be late this week or early next week, so stay tuned for that!

@mydea mydea deleted the fn/ember-auto-import branch December 13, 2021 15:01
onurtemizkan pushed a commit that referenced this pull request Dec 19, 2021
* Allow ember-auto-import 1.x or 2.x

* Update ember-cli-typescript from 3.1.4 to 4.2.1

* Update ember-cli-htmlbars from 5.7.1 to 6.0.1
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.

4 participants