Skip to content

Conversation

@bertdeblock
Copy link
Member

No description provided.

"ember-cli-babel": "^7.26.6",
"ember-cli-dependency-checker": "^3.2.0",
"ember-cli-inject-live-reload": "^2.1.0",
"ember-export-application-global": "^2.0.1",
Copy link
Member Author

Choose a reason for hiding this comment

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

Doesn't work in Ember v5.

"ember-source": "~3.28.0",
"ember-source-channel-url": "^3.0.0",
"ember-styleguide": "^5.0.0",
"ember-styleguide": "^8.4.0",
Copy link
Member Author

@bertdeblock bertdeblock Aug 24, 2023

Choose a reason for hiding this comment

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

Older version was breaking the build with newer Ember versions. Using an in-repo / internal addon might be better? Feels fragile.

@bertdeblock bertdeblock force-pushed the fix-ci branch 2 times, most recently from 4171425 to 01d6672 Compare August 24, 2023 10:02
"registry": "https://registry.npmjs.org"
},
"ember": {
"edition": "octane"
Copy link
Member Author

Choose a reason for hiding this comment

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

Required in newer Ember versions.

@kategengler
Copy link
Member

Thank you!

@kategengler kategengler merged commit 60d499b into ember-cli:master Aug 24, 2023
@bertdeblock bertdeblock mentioned this pull request Aug 25, 2023
7 tasks
@ef4 ef4 added the internal label Nov 25, 2025
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.

5 participants