Skip to content

Conversation

@imaustink
Copy link
Contributor

@imaustink imaustink commented Aug 15, 2017

Upgrade bit-docs to steal 1.x

The main goal of this PR is to upgrade steal. Testing these changes on our bit-docs generated sites exposed the need for a package override feature.

  • Upgraded steal to 1.x
  • Cleaned up tests
  • Added package override feature

This branch was based on minor, and rebased on master.

Closes #11

@imaustink imaustink force-pushed the 11-update-steal-to-1.x branch 3 times, most recently from c5e70e3 to f41dc6f Compare August 15, 2017 17:31
@imaustink imaustink requested a review from chasenlehara August 15, 2017 17:32
@imaustink imaustink force-pushed the 11-update-steal-to-1.x branch from f41dc6f to 88146e5 Compare August 15, 2017 17:34
@imaustink imaustink changed the title 11 update steal to 1.x Update steal to 1.x Aug 15, 2017
The main goal of this PR is to upgrade steal. Testing these changes on our bit-docs generated sites exposed the need for a package override feature.

- Upgraded steal to 1.x
- Cleaned up tests
- Added package override feature

This branch was based on `minor`, and rebased on `master`.

Closes #11
@imaustink imaustink force-pushed the 11-update-steal-to-1.x branch from 88146e5 to 91bc024 Compare August 15, 2017 19:28
return readFile(path.join(fullBuildFolderPath, "package.json")).then(function(packageContents){
var json = JSON.parse(packageContents);

json = _.merge(json || {}, siteConfig.html.package);
Copy link
Member

Choose a reason for hiding this comment

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

not sure where ... but it would be good to document that .dependencies overwrites .package.

@imaustink imaustink merged commit f7eb204 into master Aug 16, 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.

4 participants