Skip to content

Conversation

@ariporad
Copy link
Contributor

OK @sindresorhus, This is half of #373, and half of the fixes #370. I think the git history is pretty clean, but let me know if there's anything I should fix.

Before:
Screenshot
After:
Screenshot
Files:
Screenshot

@ariporad ariporad self-assigned this Dec 27, 2015
@vadimdemedes
Copy link
Contributor

@ariporad Could you describe what this PR improves/fixes and provide an example of before/after output?

@ariporad
Copy link
Contributor Author

@vdemedes: on mobile atm, but I'll post a screenshot when I get back. Basically, it properly displayes subdirectories with the default reporter.

api.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

This should be in the other PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll fix that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sindresorhus: Fixed.

@ariporad
Copy link
Contributor Author

@vdemedes: I uploaded some screenshots.

@ariporad ariporad force-pushed the better-test-prefixes branch from 04283ac to c56ed9e Compare December 27, 2015 18:32
@vadimdemedes
Copy link
Contributor

Great, thanks!

@sindresorhus @jamestalmage

@sindresorhus
Copy link
Member

LGTM

@ariporad
Copy link
Contributor Author

@vdemedes: Everything else look good about this PR?

@ariporad
Copy link
Contributor Author

@sindresorhus: Sorry if this is stupid, but why isn't there a LGTM status check for this PR?

@sindresorhus
Copy link
Member

@ariporad437e88f

@ariporad
Copy link
Contributor Author

@sindresorhus: Ok, thanks.

@ariporad
Copy link
Contributor Author

@sindresorhus: I'm going to merge this, is that OK?

vadimdemedes pushed a commit that referenced this pull request Dec 28, 2015
@vadimdemedes vadimdemedes merged commit e7f53a9 into avajs:master Dec 28, 2015
ariporad added a commit that referenced this pull request Dec 28, 2015
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