Skip to content

Setup coverage with undercover/coveralls#37

Open
AdrieanKhisbe wants to merge 1 commit intoecukes:masterfrom
AdrieanKhisbeArchives:setup-coverage
Open

Setup coverage with undercover/coveralls#37
AdrieanKhisbe wants to merge 1 commit intoecukes:masterfrom
AdrieanKhisbeArchives:setup-coverage

Conversation

@AdrieanKhisbe
Copy link
Copy Markdown
Contributor

No description provided.

@AdrieanKhisbe
Copy link
Copy Markdown
Contributor Author

@rejeep Just rebase the branch,
Here is a preview of the coverage:
https://coveralls.io/github/AdrieanKhisbe/espuds 😃

Comment thread test/test-helper.el
(require 'ert (f-expand "ert" espuds-test/vendor-path)))

(require 'undercover)
(undercover "espuds.el")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you need to set a report file here like you do for ecukes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

in this case nope, the default report file will suits.
We need to specify one when we need to merge the result between ecukes and ert as we do in ecukes repo

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