Skip to content
This repository was archived by the owner on Jul 2, 2026. It is now read-only.

add support for .atoum.phpstorm.php file#70

Merged
agallou merged 1 commit into
atoum:masterfrom
agallou:config_file
May 16, 2016
Merged

add support for .atoum.phpstorm.php file#70
agallou merged 1 commit into
atoum:masterfrom
agallou:config_file

Conversation

@agallou

@agallou agallou commented May 11, 2016

Copy link
Copy Markdown
Member

The plugin could now have a specific configuration file.

When the file .atoum.phpstorm.php exists on the tests root,
this config file is loaded.

Like that some configuration could be defined, for only when launching tests
from the IDE.

Example of configuration :

<?php

$script->setMaxChildrenNumber(1);
$script->disableCodeCoverage();

fixes #65

The plugin could now have a specific configuration file.

When the file .atoum.phpstorm.php exists on the tests root,
this config file is loaded.

Like that some configuration could be defined, for only when launching tests
from the IDE.

Example of configuration :
```
<?php

$script->setMaxChildrenNumber(1);
$script->disableCodeCoverage();
```
@armandabric

Copy link
Copy Markdown

PHPStorm allow to share the content of the .idea folder. Does it will be easier to add these configurations somewhere in this directory?

@agallou

agallou commented May 12, 2016

Copy link
Copy Markdown
Member Author

@Spy-Seth did you mean https://www.jetbrains.com/help/idea/2016.1/synchronizing-and-sharing-settings.html ?
It won't be easier to implement.
The .atoum.phpstorm.php could be commited and shared between developers.
If we use this sync, it would be for configuration inside phpstorm not a php file to configure it.

@armandabric

Copy link
Copy Markdown

Yes I'm talking about this. We already have a lot's of dotfiles on the root of the project. I just want to suggest to not add another one :)

@mnapoli

mnapoli commented May 13, 2016

Copy link
Copy Markdown

I tested it and confirm that it works!

@vdechenaux

Copy link
Copy Markdown
Member

👍 !

@agallou
agallou merged commit 0eebe9b into atoum:master May 16, 2016
@agallou
agallou deleted the config_file branch May 16, 2016 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set -mcn option?

4 participants