cucumber.yml is loaded/parsed twice

[jasonkarns]

The cucumber.yml file is loaded and parsed a second time when Options.parse is called from Options#merge_profiles.

[aslakhellesoy]

Is that a problem? If so, why?

[jasonkarns]

It's not really a problem, as much as a code-smell maybe? We first noticed it when we threw some info-level environment logging into our cucumber.yml. We've since moved that out. Still, it's something that clearly goes against the principle of least surprise. I would have never expected a configuration file to be parsed twice for every run.
And as google didn't find many resources describing this behavior, I thought it should at least be discussed.

[aslakhellesoy]

Feel free to send a pull request to improve this. Personally I don't care.

[dkowis]

@mattwynne @aslakhellesoy We could just close this? I don't think it's a serious bug, and I don't think I care either :)

[mattwynne]

I care. I've tried to print messages from cucumber.yml before and been irritated that they appeared twice. And I agree with the OP that it's a code smell.

[tooky]

I agree with @mattwynne, its the sort of thing that shouldn't be happening.

I have a feeling that I may have seen where this is happening when I was working on #398 - I'll take a look at this and see what I can do.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.