Skip to content

Conversation

@arings
Copy link

@arings arings commented Jul 9, 2016

Fixes / updates #898

@nfcampos
Copy link
Contributor

nfcampos commented Jul 9, 2016

that screenshot is for npm's verbose mode not ava´s. you'd have to run npm test -- --verbose to get ava's verbose mode, without the extra -- you're passing the verbose flag to npm itself

@arings
Copy link
Author

arings commented Jul 9, 2016

I am very sorry for that. AVA is new to me, and I am new to new technologies. As I understand correctly - old screenshot was sufficient, so I put it instead.

@nfcampos
Copy link
Contributor

nfcampos commented Jul 9, 2016

No need to apologise :) passing flags to npm run is rather confusing

@sindresorhus sindresorhus changed the title update docs for new reporters section + new screenshot file add new reporters section in the readme Jul 11, 2016
readme.md Outdated

<img src="media/screenshot.png" width="150">

### Optional TAP output
Copy link
Member

Choose a reason for hiding this comment

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

Tap reporter

Copy link
Member

Choose a reason for hiding this comment

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

Also need to update the link in "Why AVA?" then.

Copy link
Contributor

Choose a reason for hiding this comment

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

All caps (TAP) is the preferred representation based on the website http://testanything.org

@arings
Copy link
Author

arings commented Jul 12, 2016

'Optional TAP output' >> 'TAP reporter',
links in 'Why AVA?' updated,
Mini-reporter mentioned that it's the default reporter,
Verbose reporter mentioned that it can be selected with the --verbose flag,
Reporters section added to the Table of Contents.

@arings arings closed this Jul 12, 2016
@arings arings reopened this Jul 12, 2016
@sindresorhus sindresorhus merged commit b54eafa into avajs:master Jul 12, 2016
@sindresorhus
Copy link
Member

Looks good. Thank you @mk-61 :)

@arings arings deleted the update-docs-for-new-reporters-section branch July 12, 2016 09:45
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.

4 participants