-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactoring/aws/directconnect #222
Refactoring/aws/directconnect #222
Conversation
Codecov Report
@@ Coverage Diff @@
## refactoring/resource-configs #222 +/- ##
================================================================
+ Coverage 28.71% 28.78% +0.06%
================================================================
Files 97 98 +1
Lines 5599 5611 +12
================================================================
+ Hits 1608 1615 +7
- Misses 3991 3996 +5
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## refactoring/resource-configs #222 +/- ##
===============================================================
+ Coverage 29.63% 29.7% +0.07%
===============================================================
Files 100 101 +1
Lines 5312 5326 +14
===============================================================
+ Hits 1574 1582 +8
- Misses 3738 3744 +6
Continue to review full report at Codecov.
|
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.
Good job! Although, we need to make directconnect facade async (see #224 for context and details), I left a couple of suggestions to do so.
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.
Great work! 👌
Co-Authored-By: Aboisier <aboisiermichaud@gmail.com>
Migrated DirectConnect. See #183 .