-
-
Notifications
You must be signed in to change notification settings - Fork 46
Fix Vite version of the tutorial #272
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
base: main
Are you sure you want to change the base?
Conversation
``` | ||
|
||
```run:command hidden=true cwd=super-rentals | ||
# hack until https://github.com/emberjs/ember.js/pull/20988 is fixed |
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.
While this will make the tutorial build, anybody following it will see an error and be blocked
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'm assuming you're leaving this comment for anyone else reading along since we discussed this on discord 🤔
We're just waiting for the next beta with the backport of the linked PR to be released and I'll remove this line 👍
This looks like it's not going to be too difficult to fix 🎉
Found one blocker upstream so far: emberjs/ember.js#20988