Pylint alerts corrections as part of intervention experiment

[evidencebp]

The PR takes care of intervention for this issue as part of an experiment.
The experiment is detailed here

Code uses the pattern
if test:
return True
else:
return False

Pylint alerts on that and suggest a simplification: The if statement can be replaced with 'return bool(test)' (simplifiable-if-statement)

Hence I removed the if and instead returned the test directly.

That simplifies the code and makes it easier to understand.

[evidencebp]

Thanks, @alanhamlett !
Can you also merge it?
This way the projcect will benefit and the experimnet will be able to investigate the change implications.

[alanhamlett]

Thanks, @alanhamlett ! Can you also merge it? This way the projcect will benefit and the experimnet will be able to investigate the change implications.

I have no idea what that sentence means... Either way I don't have merge permission on this repo.

[evidencebp]

@alanhamlett ,part of the experiment design is the see if tendency to bugs, time to modify the code, etc. improve due to the change.

@yigitguler, I see that you did the recent merges in this repository.
Can you merge the PR?

[alanhamlett]

@alanhamlett ,part of the experiment design is the see if tendency to bugs, time to modify the code, etc. improve due to the change.

@yigitguler, I see that you did the recent merges in this repository. Can you merge the PR?

Waste of time

[evidencebp]

It is a minor change.
I don't expect it to have a huge influence on the file.
However, quantifying the mederate (or even lack of) benefit of alerts is an importnat contribution for SE research.

And since the PR is already written and approved, just merging it require almost no time.
This way the time that we already invested won't be wasted ;-)

[yigitguler]

Lets see what happens.

[yigitguler]

To me, the difference should be negligible.

[alanhamlett]

Thanks @evidencebp