-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add and update documentation (#8 / #40) #40
Conversation
Bukama
commented
Mar 27, 2021
•
edited
Loading
edited
@@ -71,9 +71,11 @@ This example shows a report which include: | |||
Several values are configurable by system properties. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why system properties? Wouldn't a testreport.properties
file (in the test resources) make more sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be file-independent and be able to use the SystemPropertyExtension
:)
docs/project-page.adoc
Outdated
|
||
The `Test Reports` provides additional information in (new) reports of the test results. | ||
The project was initialized as the current JUnit test report can not publish additional attributes or information, e.g. values created by extensions. | ||
The library itself does not create one single report, but provides test report provide the user can use. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is not clear. Does the library create test reports or not? What is a "test report provider"?
Also, I think you meant to write "but provides test report provider(s)"?
Thanks @Michael1993 Updated the parts you mentioned |