Skip to content

Prepare for FastBoot 1.0#20

Merged
kehphin merged 2 commits intomasterfrom
prepare-fastboot-1.0
Jun 30, 2017
Merged

Prepare for FastBoot 1.0#20
kehphin merged 2 commits intomasterfrom
prepare-fastboot-1.0

Conversation

@kehphin
Copy link

@kehphin kehphin commented Jun 30, 2017

In Fastboot 1.0, we no longer have access to EMBER_CLI_FASTBOOT.

Resolution:

Leverage fastboot-transform to keep ember-cli-sticky fastboot compliant (doing a if (typeof FastBoot === 'undefined') check under the covers)

@kehphin kehphin requested a review from dfreeman June 30, 2017 17:30
Copy link
Member

@dfreeman dfreeman left a comment

Choose a reason for hiding this comment

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

One little comment, otherwise looks good. Thanks @kehphin!

import: ['jquery.sticky.js'],
processTree(input) {
return fastbootTransform(input);
}
Copy link
Member

Choose a reason for hiding this comment

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

This could be simplified a little now that it's not relying on process.env to something like:

'jquery-sticky': {
  import: {
    include: ['jquery.sticky.js'],
    processTree(input) {
      return fastbootTransform(input);
    }
  }
}

@kehphin kehphin merged commit 725595d into master Jun 30, 2017
@kehphin
Copy link
Author

kehphin commented Jun 30, 2017

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