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.

Sign up for GitHub

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

Jump to bottom

XX-MAXCOVERAGE-XX is not replaced by its value in the docx report #244

Closed
ErwanGauduchon opened this issue May 11, 2021 · 0 comments · Fixed by #245 or #279
Closed

XX-MAXCOVERAGE-XX is not replaced by its value in the docx report #244

ErwanGauduchon opened this issue May 11, 2021 · 0 comments · Fixed by #245 or #279
Assignees
@ErwanGauduchon
Labels
bug Something isn't working done This issue was solved but not merged in master
Milestone
4.0.0

Comments

@ErwanGauduchon
Copy link
Contributor

Describe the bug

When generating a report, XX-MAXCOVERAGE-XX appears in the metrics table of the docx report while there should be a value.

To reproduce

Generate the report, open the docx file and go to the metrics table.

Expected behavior

XX-MAXCOVERAGE-XX should be replaced by the same value as in the markdown file.
@ErwanGauduchon ErwanGauduchon added the bug Something isn't working label May 11, 2021
@ErwanGauduchon ErwanGauduchon added this to the 4.0.0 milestone May 11, 2021
@ErwanGauduchon ErwanGauduchon self-assigned this May 11, 2021
ErwanGauduchon added a commit that referenced this issue May 12, 2021
@ErwanGauduchon
fix replacement of XX-MAXCOVERAGE-XX (#244)
c9b200e
@ErwanGauduchon ErwanGauduchon mentioned this issue May 12, 2021
Merged
12 tasks
@Sancretor Sancretor linked a pull request May 12, 2021 that will close this issue
Fix replacement of XX-MAXCOVERAGE-XX #245
Merged
12 tasks
@Sancretor Sancretor added the done This issue was solved but not merged in master label May 12, 2021
@Sancretor Sancretor mentioned this issue Aug 27, 2021
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ErwanGauduchon ErwanGauduchon

Labels
bug Something isn't working done This issue was solved but not merged in master
Projects
None yet
Milestone
4.0.0
2 participants
@Sancretor @ErwanGauduchon