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

No longer provide a feature in test-helper.el #25

Merged
merged 1 commit into from
Apr 11, 2017
Merged

No longer provide a feature in test-helper.el #25

merged 1 commit into from
Apr 11, 2017

Conversation

tarsius
Copy link
Contributor

@tarsius tarsius commented Apr 11, 2017

The file isn't a library intended to be loaded with require.
Instead ert-runner loads it using load. Other packages that
use ert-runner also come with a file named test-helper.el and
unfortunately many of those also provide test-helper, which
leads to conflicts.

Fixes #24.

The file isn't a library intended to be loaded with `require`.
Instead `ert-runner` loads it using `load`.  Other packages that
use `ert-runner` also come with a file named test-helper.el and
unfortunately many of those also provide `test-helper`, which
leads to conflicts.

Fixes #24.
@coveralls
Copy link

coveralls commented Apr 11, 2017

Coverage Status

Coverage remained the same at 90.244% when pulling c776944 on tarsius:master into 2b966ed on johanvts:master.

@AdrieanKhisbe AdrieanKhisbe merged commit 23a444f into johanvts:master Apr 11, 2017
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.

None yet

3 participants