fix: Use correct field for evaluating whether or not GitHub Enterprise is selected #5987
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e is selected Signed-off-by: Jonah Back <[email protected]>
backjo
force-pushed
the
fix/GithubEnterprise
branch
from
39d259b to
6c2d692
Compare
alexmt
approved these changes
Apr 7, 2021
There was a problem hiding this comment.
Thank you @backjo !
LGTM
PR was tested using https://gitpod.io/#https://github.com/backjo/argo-cd/tree/fix/GithubEnterprise 👯
Codecov Report
@@ Coverage Diff @@
## master #5987 +/- ##
==========================================
+ Coverage 40.95% 40.96% +0.01%
==========================================
Files 147 147
Lines 19656 19656
==========================================
+ Hits 8051 8053 +2
+ Misses 10499 10498 -1
+ Partials 1106 1105 -1
Continue to review full report at Codecov.
|
alexmt
pushed a commit
that referenced
this pull request
Apr 12, 2021
…e is selected (#5987) Signed-off-by: Jonah Back <[email protected]>
yujunz
added a commit
to abcue/argo-cd
that referenced
this pull request
Apr 15, 2021
5bc7297 fix: bitbucket server failing diagnostics:ping (argoproj#6029) (argoproj#6034) 8f53bd5 fix: add helm dependencies with custom CA (argoproj#6003) 8fd6f13 docs: Custom resource actions (argoproj#5838) 8a2897d docs: update delete policy verbiage (argoproj#6025) c847bd9 chore: remove Argo CD CRDs from namespaced install (argoproj#6022) 61080b3 docs: improve Orphaned Resources Monitoring with more examples and correct grammar (argoproj#6006) 8301d39 Adding explicit bind to redis and sentinel for IPv4 clusters argoproj#5957 (argoproj#6005) 12cabdf fix: adding tests for helm OCI registry (argoproj#5978) 9da9514 docs: Add Ant Group to the list of users (argoproj#6011) 5e34a8a add Polarpoint.io (argoproj#6010) 2f92777 chore: move access checks from api server to repo server (argoproj#5940) ae2d0ff fix(ui): Unscheduled pods in node view are now visible. Fixes argoproj#5981 (argoproj#5988) b003f70 docs: SealedSecret status missing on k8s 1.16+ (argoproj#5846) 445872f fix: use correct field for evaluating whether or not GitHub Enterprise is selected (argoproj#5987) 9afa833 chore: Make e2e tests runnable against remote cluster (argoproj#5895)
shubhamagarwal19
pushed a commit
to shubhamagarwal19/argo-cd
that referenced
this pull request
Apr 15, 2021
…e is selected (argoproj#5987) Signed-off-by: Jonah Back <[email protected]>
jannfis
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #5986
Signed-off-by: Jonah Back [email protected]
Checklist: