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

fix: made envs application actually NOP #691

Open
wants to merge 3 commits into
base: main-enterprise
Choose a base branch
from

Conversation

randomcascade
Copy link

@randomcascade randomcascade commented Sep 24, 2024

Fixes: #551 Would love if someone else could validate the testing as I'm not sure I've tested every case.

@luvsaxena1
Copy link
Contributor

@decyjphr Another contribution from my company, We noticed this is an issue which people does not noticed and it impact environments.

@klutchell
Copy link
Contributor

Any objection to this one @decyjphr ? I would love to see it merged.

@randomcascade
Copy link
Author

It looks like plugins have received an update to include an errors parameter so this plugin may not be 100% in line, if that's a problem I can update it sometime in the next week or so.

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.

Nop on PRs doesn't work for environments
3 participants