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

Fix build for ember-source 4.0+ #396

Merged
merged 2 commits into from
Nov 17, 2022
Merged

Conversation

wagenet
Copy link
Contributor

@wagenet wagenet commented Nov 11, 2022

Fixes #393.

...or at least attempts to!

@mixonic
Copy link
Contributor

mixonic commented Nov 12, 2022

FYI CI on the repo is very bad. I may need to rerun failures to get then green, if they don't look related.

@runspired
Copy link
Contributor

Best I can tell CI failures are unrelated for most of the versions but we may have broken 3.12/3.16/3.20

@runspired runspired marked this pull request as ready for review November 17, 2022 19:24
@runspired runspired merged commit 1317065 into html-next:master Nov 17, 2022
@runspired runspired added the bug label Nov 17, 2022
@runspired runspired changed the title Fix build for 4.0 Fix build for ember-source 4.0+ Nov 17, 2022
@mixonic
Copy link
Contributor

mixonic commented Nov 18, 2022

Uh. I hope we did not? Those versions are supported and the support is used by ember table and apps I work on.

@mixonic
Copy link
Contributor

mixonic commented Nov 18, 2022

3.12, 16, and 20 are green in the builds. Why do you think they may be broken?

@runspired
Copy link
Contributor

@mixonic you missed I added a commit to fix them

@mixonic
Copy link
Contributor

mixonic commented Nov 18, 2022

Ah, amazing. Thanks @runspired @wagenet !

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

Successfully merging this pull request may close these issues.

Transpiling error in Ember 4 without embroider
3 participants