-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
Add ability to get it working with proxy #144
Conversation
@8398a7 can you please take a look at this PR? |
@sethumadhav07 Sorry for the late confirmation. |
# Conflicts: # package-lock.json # package.json
@8398a7 can you please take a look at it and approve? |
Codecov Report
@@ Coverage Diff @@
## master #144 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 2 2
Lines 204 209 +5
Branches 50 52 +2
=========================================
+ Hits 204 209 +5
Continue to review full report at Codecov.
|
CI is failing due to reduced test coverage. Can you please fix it? |
I fixed the code coverage for tests. can you please take a look at it and approve? |
@8398a7 can a new release/tag be created that involves this change? |
currently slack connection using webhook is not working with proxy which is a common requirement in many corporate settings.
Code change in this PR addresses that.