Skip to content

Nested folder for ember coverage-merge - #93

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

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

Conversation

@designblooz

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

@rwwagner90 fixed failing test

@RobbieTheWagner

Copy link
Copy Markdown
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