Skip to content
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

Remove blueprints for view and view-test #111

Closed
wants to merge 1 commit into from

Conversation

mriska
Copy link
Contributor

@mriska mriska commented Jun 8, 2016

They are removed in ember-cli as of 2.4.3 ember-cli/ember-cli#5607
And having them in ember-cli-coffeescript gives errors if using a never version of ember-cli in the same project

Cannot find module 'ember-cli/blueprints/view'
Error: Cannot find module 'ember-cli/blueprints/view'
  at Function.Module._resolveFilename (module.js:339:15)
  at Function.Module._load (module.js:290:25)
  at Module.require (module.js:367:17)
  at require (internal/module.js:20:19)
  at Object.<anonymous> (/Users/mriska/work/test3/node_modules/ember-cli-coffeescript/blueprints/view/index.js:1:80)

They are removed in ember-cli as of 2.4.3 ember-cli/ember-cli#5607
And having them in ember-cli-coffeescript gives errors if using a never version of ember-cli in the same project

```
Cannot find module 'ember-cli/blueprints/view'
Error: Cannot find module 'ember-cli/blueprints/view'
  at Function.Module._resolveFilename (module.js:339:15)
  at Function.Module._load (module.js:290:25)
  at Module.require (module.js:367:17)
  at require (internal/module.js:20:19)
  at Object.<anonymous> (/Users/mriska/work/test3/node_modules/ember-cli-coffeescript/blueprints/view/index.js:1:80)
```
@kimroen
Copy link
Owner

kimroen commented Aug 17, 2016

This isn't mergeable now because of some other changes, but I will make sure I merge it in manually before making another release. Thank you!

@kimroen
Copy link
Owner

kimroen commented Oct 3, 2016

The view-blueprints are actually still there, but they are moved to ember-cli/ember-cli-legacy-blueprints. Because of this, I am going to keep them in ember-cli-coffeescript for the time being. I'm releasing a new version today that fixes the current problem when trying to run them.

Thank you again very much for taking the time to make this PR, but I am going to close it now. Hopefully this makes sense.

@kimroen kimroen closed this Oct 3, 2016
@mriska
Copy link
Contributor Author

mriska commented Oct 3, 2016

No problem, thanks for fixing the issue when running them.

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.

None yet

2 participants