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

ci: add code coverage #45

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

ci: add code coverage #45

wants to merge 3 commits into from

Conversation

tpluscode
Copy link
Member

No description provided.

nicky508 and others added 3 commits July 17, 2019 16:05
Added jacoco based on the blog but instead of using the test phase, used
the package phase.
@tpluscode tpluscode requested a review from nicky508 July 26, 2019 06:04
@tpluscode
Copy link
Member Author

@nicky508 would you please have another look? I fixed the styling and I notice that the coverage report is not being produced.

@nicky508
Copy link
Collaborator

@nicky508 would you please have another look? I fixed the styling and I notice that the coverage report is not being produced.

Take a look

@nicky508
Copy link
Collaborator

I think I've found probably some error. I am not able to build the the stuff because I got this weird sign everywhere in the code "Ã". I removed them but the test still do not run since I got this outcome:

{DEFAULT_OUTPUT__synthetic0.api.json={
    "steps": [
        �FOR block:blocks SEPARATOR ","�
            �block.step�
        �ENDFOR�
    ]
}
}

Looks like this weird sign again. I am not sure where this sign is coming from.

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