Skip to content

Comments

Nested folder for ember coverage-merge#93

Merged
RobbieTheWagner merged 2 commits intoember-cli-code-coverage:masterfrom
designblooz:nested-coverage-folder
Sep 29, 2017
Merged

Nested folder for ember coverage-merge#93
RobbieTheWagner merged 2 commits intoember-cli-code-coverage:masterfrom
designblooz:nested-coverage-folder

Conversation

@designblooz
Copy link
Contributor

Not sure if this is intended or not, but ember coverage-merge does not work when coverageFolder is nested.

matchDir option for node-dir only works for direct child folders of root folder.

@designblooz designblooz changed the title [Bug Fix] Nested folder for ember coverage-merge Nested folder for ember coverage-merge Jan 12, 2017
@designblooz designblooz force-pushed the nested-coverage-folder branch 3 times, most recently from ddc6013 to 673833c Compare January 12, 2017 17:23
@RobbieTheWagner
Copy link
Collaborator

@designblooz sorry for the delay in reviewing this! I merged master into this PR and the tests appear to be failing. Would you mind taking a look please?

@designblooz designblooz force-pushed the nested-coverage-folder branch from 691c214 to 307f31e Compare September 27, 2017 14:44
@designblooz designblooz force-pushed the nested-coverage-folder branch from 307f31e to 3a702aa Compare September 27, 2017 14:45
@designblooz
Copy link
Contributor Author

@rwwagner90 fixed failing test

@RobbieTheWagner
Copy link
Collaborator

Thanks @designblooz!

@RobbieTheWagner RobbieTheWagner merged commit 5155112 into ember-cli-code-coverage:master Sep 29, 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