-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
Lint everything #712
Lint everything #712
Conversation
4625bca
to
8949ad1
Compare
8949ad1
to
dd3c3b4
Compare
e4bbb9e
to
6654542
Compare
@NullVoxPopuli What do you think which versions of ember should we support? As you already mentioned here: #711 (comment) the new release should be a major bump. |
yup, sounds good to me. I don't think it makes sense to support ember pre-octane |
d1c9640
to
06a4477
Compare
06a4477
to
cdebc1d
Compare
@NullVoxPopuli I updated the PR description ^^ and would suggest that I open a followup PR with the changes to the template transform (which I already did) to keep this PR focused. |
@NullVoxPopuli Thanks for merging. Wanna add the internal label? |
This is a followup on #711 to get CI green.
Most of the commit messages should be self explanatory and it should be easy to review commit by commit.
Failing scenarios: The release, beta and canary runs still fail. The error is:
and is at least caused by the
htmlbars-plugins/v-get.js
template transform and maybe other dependencies.I would suggest that I open a followup PR with the changes to the template transform (which I already did - 7f98630) to keep this PR focused.