Skip to content

[WIP] Update ember-cli#16459

Closed
GavinJoyce wants to merge 4 commits intoemberjs:masterfrom
GavinJoyce:gj/try-latest-ember-cli
Closed

[WIP] Update ember-cli#16459
GavinJoyce wants to merge 4 commits intoemberjs:masterfrom
GavinJoyce:gj/try-latest-ember-cli

Conversation

@GavinJoyce
Copy link
Member

@GavinJoyce GavinJoyce commented Apr 3, 2018

This pins ember-cli to include ember-cli/ember-cli#7731 so that we can implement more generator blueprints in ember-cli/ember-cli#7530

TODO:

  • Fix remaining 16 failing tests

@GavinJoyce
Copy link
Member Author

GavinJoyce commented Apr 3, 2018

destroyApp has been removed: ember-cli/ember-cli@ec12c75. We'll can remove a lot of dead code in the blueprints and supporting files

related RFCs:

testFolderRoot = pathUtil.getRelativeParentPath(options.entity.name, -1, false);
}

let destroyAppExists = fs.existsSync(
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, I think this still needs to exist no? This isn't about the ember-cli version that is being used by ember-source, but rather about the current project layout when the blueprint is ran...

Copy link
Member Author

Choose a reason for hiding this comment

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

You're right, thanks for pointing this out. I enjoy deleting code so much, I just jumped right in 🙈

@GavinJoyce
Copy link
Member Author

I'm going to close this and revisit in the morning.

@GavinJoyce GavinJoyce closed this Apr 3, 2018
@GavinJoyce GavinJoyce deleted the gj/try-latest-ember-cli branch April 3, 2018 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants