-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
Upgrade ember-cli-addon-docs and enable Ember v4 scenarios #747
Conversation
@@ -37,6 +37,7 @@ module.exports = async function () { | |||
name: 'ember-lts-3.12', | |||
npm: { | |||
devDependencies: { | |||
'ember-cli': '~3.12.0', |
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.
With latest Ember CLI there is a build time error originated from ember-cli-addon-docs
@@ -78,6 +79,7 @@ module.exports = async function () { | |||
npm: { | |||
devDependencies: { | |||
'ember-source': await getChannelURL('beta'), | |||
'ember-template-lint': '^3.10.0', |
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.
with ember-template-lint@2
there is build time error in beta scenario
@@ -43,7 +43,7 @@ | |||
"cli-table3": "^0.6.0", | |||
"debug": "^4.2.0", | |||
"ember-auto-import": "^2.2.3", | |||
"ember-cli-babel": "^7.21.0", | |||
"ember-cli-babel": "^7.26.6", |
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.
This fixes build time warning about old ember-cli-babel
version
@@ -61,8 +61,8 @@ | |||
"babel-eslint": "^10.1.0", | |||
"codeclimate-test-reporter": "^0.5.0", | |||
"ember-cli": "^3.28.3", | |||
"ember-cli-addon-docs": "^0.10.0", | |||
"ember-cli-addon-docs-yuidoc": "^0.2.4", | |||
"ember-cli-addon-docs": "^4.0.3", |
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.
had to be upgraded to make CI pass with Ember v4 beta/canary (there is bunch if fixes in transitive dependencies)
this was done in #768 |
No description provided.