Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test & readme #8

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Test & readme #8

wants to merge 7 commits into from

Conversation

Grummfy
Copy link
Member

@Grummfy Grummfy commented Aug 12, 2017

  • adding unit test
  • adding behaviour test
  • fix not found extension bug
  • adding readme

README.md Outdated
@@ -0,0 +1,34 @@
This repository is the source files for [http://extensions.atoum.org](http://extensions.atoum.org).

The purpose is to list extension of atoum and expose it.
Copy link
Member

Choose a reason for hiding this comment

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

s/extension.extensions ?


extensions:
Behat\MinkExtension:
goutte: ~
Copy link
Member

Choose a reason for hiding this comment

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

remove the comments ?

README.md Outdated
### Test

```
vendor/bin/robo test
Copy link
Member

Choose a reason for hiding this comment

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

where is the robo test command ?

Copy link
Member Author

Choose a reason for hiding this comment

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

oups I forget to remove it, I added it, then see that robot need to be updated then see that your libs doesn't support to be updated so roll it back ;)

@Grummfy
Copy link
Member Author

Grummfy commented Sep 8, 2017

thanks for the review, here is the fix ;)

@Grummfy
Copy link
Member Author

Grummfy commented Oct 2, 2017

@agallou tu voix d'autres changements?

@Grummfy
Copy link
Member Author

Grummfy commented Mar 4, 2018

@agallou seems ready

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