Skip to content

Conversation

DavertMik
Copy link
Member

No description provided.


If you want to use it as default reporter enable it in `codeception.yml`.

But what if you need to change the output format of the XML or JSON results triggered with the `--xml` or `--json` options?
Copy link
Member

Choose a reason for hiding this comment

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

Text about custom PHPUnit printers must be removed because they are no longer supported in Codeception 5.

Copy link
Member

Choose a reason for hiding this comment

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

oh, this change will break PHPStorm integration, because they use -o "reporters: report: PhpStorm_Codeception_ReportPrinter"

Copy link
Member

Choose a reason for hiding this comment

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


| Event | When? | Triggered by
|:--------------------:| --------------------------------------- | --------------------------:
| `suite.before` | Before suite is executed | [Suite, Settings](https://github.com/Codeception/Codeception/blob/4.0/src/Codeception/Event/SuiteEvent.php)
Copy link
Member

Choose a reason for hiding this comment

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

Many links point to 4.0 branch

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@Naktibalda
Copy link
Member

It would be great if you could document in https://codeception.com/docs/06-ModulesAndHelpers#Dynamic-Configuration-With-Parameters that "%NAME%" placeholder can be replaced not only with string value, but with object, array, float, bool or null too. See Codeception/Codeception#6409

}

{% endhighlight %}

Copy link
Member

Choose a reason for hiding this comment

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

  1. Why do we have 2 copies of all docs files now? If you are renaming, it would be better to replace content of the old page with meta or javascript redirect to the new place.
  2. I see that atributes are not documented in AdvancedUsage page.

Copy link
Member

@Naktibalda Naktibalda left a comment

Choose a reason for hiding this comment

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

Please resolve conflicts in js/scripts.js and replace old guide files with symlinks or redirects.

Example of redirecting file: https://raw.githubusercontent.com/Codeception/codeception.github.com/master/for/symfony.md
Symlink to new module: https://github.com/Codeception/codeception.github.com/blob/master/docs/modules/ZF2.md

@DavertMik DavertMik merged commit 51d2387 into master Jul 28, 2022
@Naktibalda Naktibalda deleted the 5.0 branch July 29, 2022 18:49
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