skipOn should respect environment name

[bahmutov]

Not just isOn

// CYPRESS_ENVIRONMENT=draft
skipOn('draft', () => { ... })

[rafael-anachoreta]

Hey @bahmutov , are you accepting PRs for this issue? I've stumbled upon it and would love to have it fixed.

[bahmutov]

Sure would love a pull request

…

Sent from my iPhone

On Mar 11, 2020, at 06:26, Rafael Anachoreta ***@***.***> wrote:  Hey @bahmutov , are you accepting PRs for this issue? I've stumbled upon it and would love to have it fixed. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[Darex1991]

hey @bahmutov any news? I would love to have this :)

[rafael-anachoreta]

That's my bad, I was meant to have submitted the PR. I'll get to it later today.

[rafael-anachoreta]

@bahmutov I've created the PR above to address this issue. Let me know your thoughts!

[unikitty37]

Is there any more news on this?

I can skip using cy.skipOn(Cypress.env('ENVIRONMENT') === 'whatever'), but it leaves a rather cryptic log message of skipOn false rather than skipOn whatever, so it's not ideal for readability…