Issue with pytest_collect_ignore implementation

[bluetech]

Hi, pytest developer here, coming from here.

This plugin implements the pytest_collect_ignore hook here:

pytest-repo-health/pytest_repo_health/plugin.py

Lines 140 to 149 in 642346d

	def pytest_ignore_collect(path, config):
	"""
	pytest hook that determines if pytest looks at specific file to collect tests
	if repo_health is set to true:
	only tests in test files with "repo_health" in their path will be collected
	"""
	if config.getoption("repo_health"):
	if "/repo_health" not in str(path):
	return True
	return False

This hook has "firstresult" semantics, which means that if a plugin returns a non-None result, this is the result and further processing is stopped. This plugin does return False in the non-ignored case, but due to firstresult what it should do is return None to allow other plugins (including pytest itself) to perform their ignores.