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

Ember data dependency #242

Closed
ryanto opened this issue Sep 10, 2018 · 2 comments
Closed

Ember data dependency #242

ryanto opened this issue Sep 10, 2018 · 2 comments

Comments

@ryanto
Copy link
Contributor

ryanto commented Sep 10, 2018

I have an issue I'm running into that I'm not quite sure how to solve. I'll try my best to explain, it's a little fuzzy exactly what's going on.

I'm writing an addon that is designed to work alongside Ember Data. On travis, my Ember try config lists multiple versions of Ember and Data. For example...

// config/ember-try.js

// ...
{
  name: 'ember-lts-2.12',
  npm: {
    devDependencies: {
      'ember-source': '~2.12.0',
      'ember-data': "~2.12.0"
    }
  }
},
// ...
{
  name: 'ember-3.4',
  npm: {
    devDependencies: {
      'ember-source': '~3.4.0',
      'ember-data': "~3.4.0"
    }
  }
},
// ...

However, the versions of data listed above are not the versions of data that end up in my apps dummy build. It looks like the ED that ultimately ends up in the build is the version that comes from addon docs.

For example, when I run the ember-lts-2.12 scenario, I end up with an application that has addon docs 0.5.3 and Ember Data 2.18.4.

When I run yarn list I see...

yarn list ember-data
├─ [email protected]
│  └─ [email protected]
└─ [email protected]

And when I run DS.Version in the browser I see

DS.VERSION
=> "2.18.4"

Next, when I run the ember-3.4 scenario, I end up with an application that also has addon docs 0.5.3 and Ember Data 2.18.4.

yarn list ember-data
├─ [email protected]                                                                                       │
│  └─ [email protected]                                                                                             │
└─ [email protected]

And in the browser...

DS.VERSION
=> "2.18.4"

I'm wondering if you would be open to loosening the ember data dependency? Would 2.x - 3.x be ok?

This was referenced Sep 10, 2018
@ryanto
Copy link
Contributor Author

ryanto commented Sep 11, 2018

PR here: #243

@samselikoff
Copy link
Contributor

#243 closes

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

No branches or pull requests

2 participants