Skip to content
This repository has been archived by the owner on Apr 17, 2020. It is now read-only.

Allow usage of blueprint within a blueprinted project. #31

Merged

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Apr 11, 2017

Prior to these changes running ember init -b @glimmer/blueprint would always download the latest release from npm's registry for @glimmer/blueprint. This means that it is impossible to actually control your upgrading process.

This change makes this package both a stand alone blueprint package and an addon. The addon creates a blueprint for @glimmer/blueprint so that now ember init -b @glimmer/blueprint will use whatever version of @glimmer/blueprint that you happen to have installed. This mirrors the way ember-cli upgrades work (where you first update the version of ember-cli in the project, then you run ember init with that version).

Ultimately, this commit should be able to be revertted once the upstream issue is resolved: ember-cli/ember-cli#6952.

Prior to these changes running `ember init -b @glimmer/blueprint`
would always download the `latest` release from `npm`'s registry for
`@glimmer/blueprint`. This means that it is impossible to actually
control your upgrading process.

This change makes this package *both* a stand alone blueprint package
*and* an addon. The addon creates a blueprint for `@glimmer/blueprint`
so that now `ember init -b @glimmer/blueprint` will use whatever version
of `@glimmer/blueprint` that you happen to have installed. This mirrors
the way ember-cli upgrades work (where you first update the version of
ember-cli in the project, then you run `ember init` with that version).

Ultimately, this commit should be able to be revertted once the upstream
issue is resolved: ember-cli/ember-cli#6952.
@locks
Copy link
Contributor

locks commented Apr 12, 2017

This is ingenious and useful, sounds good to me!

Copy link
Contributor

@locks locks left a comment

Choose a reason for hiding this comment

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

Looks fine technically!

@rwjblue rwjblue merged commit 130915c into glimmerjs:master Apr 12, 2017
@rwjblue rwjblue deleted the make-ember-init-use-installed-version branch April 12, 2017 13:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Project#blueprintLookupPaths should account for ember-blueprint dependencies.
2 participants