-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fixes/Improvements following helm walkthrough #2 #302
Conversation
richard-cox
commented
Jan 31, 2020
•
edited
Loading
edited
- blocked on Fix stale pods #298
- fixes V3 walkthrough #2 issues #299
- Since cloudfoundry@7c94858 we take action.entity as paramount - In multi action lists for the secondary type this was incorrectly set to the primary type
Codecov Report
@@ Coverage Diff @@
## master #302 +/- ##
=========================================
- Coverage 59.06% 59% -0.06%
=========================================
Files 1027 1029 +2
Lines 31880 32059 +179
Branches 4359 4388 +29
=========================================
+ Hits 18829 18917 +88
- Misses 13051 13142 +91 |
return; | ||
} | ||
|
||
obs.pipe(delay(999999999)).subscribe(() => { |
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.
Remove delay
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.
One debug delay to remove
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