Skip to content

Conversation

kumar303
Copy link
Contributor

Fixes #332

@rpl
Copy link
Member

rpl commented Jun 30, 2016

@kumar303 r+

Nit: we could probably add one more test to assert that path.normalize is applied to both the source dir and the artifact dir paths.

@kumar303
Copy link
Contributor Author

Sure. Added.

@coveralls
Copy link

coveralls commented Jun 30, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling bf266d8 on kumar303:fix-source-dir into 0a277d9 on mozilla:master.

@kumar303 kumar303 merged commit 8d3379a into mozilla:master Jun 30, 2016
@kumar303 kumar303 deleted the fix-source-dir branch June 30, 2016 15:10
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