Skip to content

Feature/replace common utils#659

Merged
bmuenzenmeyer merged 3 commits intodev-3.0from
feature/replace-common-utils
May 7, 2017
Merged

Feature/replace common utils#659
bmuenzenmeyer merged 3 commits intodev-3.0from
feature/replace-common-utils

Conversation

@bmuenzenmeyer
Copy link
Member

Testing a merge against the CI

Summary of changes:

  • Changes plutils.emptyDirectory → fs.emptyDirSync
  • Changes plutils.mergeData → _.merge
  • Changes plutils.shuffle → _.shuffle
  • Changes plutils.isEmptyObject → _.empty

Replace plutils.mergeData with _.merge in core/lib/pseudopattern_hunter.js

Replace plutils.mergeData with _.merge in core/lib/list_item_hunter.js

Replace plutils.mergeData with _.merge in core/lib/patternlab.js

Replace plutils.isObjectEmpty with _.empty in core/lib/pattern_assembler.js

Replace plutils.shuffle with _.shuffle in core/lib/pattern_assembler.js

Replace util.emptyDirectory with fs.emptyDirSync in core/lib/starterkit_manager.js
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 64.142% when pulling 6a9674b on feature/replace-common-utils into 4eff3f2 on dev-3.0.

1 similar comment
@coveralls
Copy link

coveralls commented May 5, 2017

Coverage Status

Coverage increased (+0.4%) to 64.142% when pulling 6a9674b on feature/replace-common-utils into 4eff3f2 on dev-3.0.

@raphaelokon
Copy link
Contributor

Cheers for the test. I fixed a minor error … passing now.

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.

3 participants