Skip to content

Fix regression to the type of object that was being passed to assets adapters as “config”.#63

Merged
LevelbossMike merged 1 commit intoember-cli-deploy:masterfrom
yapplabs:62-config-refression
Mar 1, 2015
Merged

Fix regression to the type of object that was being passed to assets adapters as “config”.#63
LevelbossMike merged 1 commit intoember-cli-deploy:masterfrom
yapplabs:62-config-refression

Conversation

@lukemelia
Copy link
Copy Markdown
Contributor

Fix regression to the type of object that was being passed to assets adapters as “config”. Closes #62

  • Slight refactoring to AssetsTask to make it’s interaction with AssetsUploader more testable
  • this change means we are not being as lazy with default values as before. That’s ok.
  • ConfigurationReader now needs to be constructed with a project reference

@achambers
Copy link
Copy Markdown
Member

@lukemelia I merged PR #64 first, so might be worth a rebase of this one.

…adapters as “config”. Closes ember-cli-deploy#62

* Slight refactoring to AssetsTask to make it’s interaction with AssetsUploader more testable
* this change means we are not being as lazy with default values as before. That’s ok.
* ConfigurationReader now needs to be constructed with a project reference
@lukemelia lukemelia force-pushed the 62-config-refression branch from ada0bec to a8afaa3 Compare March 1, 2015 14:03
@lukemelia
Copy link
Copy Markdown
Contributor Author

Rebased

LevelbossMike added a commit that referenced this pull request Mar 1, 2015
Fix regression to the type of object that was being passed to assets adapters as “config”.
@LevelbossMike LevelbossMike merged commit 702d0d4 into ember-cli-deploy:master Mar 1, 2015
@LevelbossMike
Copy link
Copy Markdown
Member

👍

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.

Regression in how config is exposed to adapters

3 participants