-
Notifications
You must be signed in to change notification settings - Fork 5k
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: pass env when getting param announcement (#11812) #11815
fix: pass env when getting param announcement (#11812) #11815
Conversation
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
/cherry-pick 2.6 |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #11815 +/- ##
=======================================
Coverage 47.79% 47.80%
=======================================
Files 246 246
Lines 41968 41969 +1
=======================================
+ Hits 20058 20062 +4
+ Misses 19910 19908 -2
+ Partials 2000 1999 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/cherry-pick release-2.6 |
* fix: pass env when getting param announcement (#11812) Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * use same method as other methods Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * better tests Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * make sure env var tests are meaningful Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
* fix: pass env when getting param announcement (#11812) * use same method as other methods * better tests * make sure env var tests are meaningful --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…roj#11815) * fix: pass env when getting param announcement (argoproj#11812) Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * use same method as other methods Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * better tests Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * make sure env var tests are meaningful Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Fixes #11812