Skip to content

Fix coverage report submission failure#294

Merged
cogmission merged 1 commit intonumenta:masterfrom
cogmission:godin_coverage_fix
Aug 28, 2015
Merged

Fix coverage report submission failure#294
cogmission merged 1 commit intonumenta:masterfrom
cogmission:godin_coverage_fix

Conversation

@cogmission
Copy link
Copy Markdown
Collaborator

Fixes #293
Fixes #266

Fixed by @Godin (Evgeny Mandrikov)

@cogmission
Copy link
Copy Markdown
Collaborator Author

@rhyolight This is me checking in work from #293

I didn't get an answer regarding signing the CL so I'm committing this which fixes coverage reports!

cogmission added a commit that referenced this pull request Aug 28, 2015
Fix coverage report submission failure
@cogmission cogmission merged commit 94b1f7b into numenta:master Aug 28, 2015
@Godin
Copy link
Copy Markdown

Godin commented Aug 28, 2015

@cogmission I was offline 😆 but that's not such a big contribution to bother with reading of CLA, I'm glad that now JaCoCo works for you and there is possibility to dig deeper into JVM bug to get it fixed and that's enough.

@cogmission
Copy link
Copy Markdown
Collaborator Author

@Godin It's cool. Thank you very much for your contribution! It was small but very meaningful! :-)

@cogmission cogmission deleted the godin_coverage_fix branch August 29, 2015 16:05
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