-
Notifications
You must be signed in to change notification settings - Fork 8
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
test: remove redundent config tests #156
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @ArniStarkware and the rest of your teammates on |
0ff40fa
to
31d798d
Compare
da2d051
to
67eba04
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #156 +/- ##
==========================================
- Coverage 50.59% 49.88% -0.72%
==========================================
Files 21 21
Lines 840 840
Branches 840 840
==========================================
- Hits 425 419 -6
- Misses 394 398 +4
- Partials 21 23 +2 ☔ View full report in Codecov by Sentry. |
31d798d
to
9dbd6d0
Compare
67eba04
to
166d248
Compare
9dbd6d0
to
26e9410
Compare
166d248
to
43fc7d4
Compare
26e9410
to
c8a1e02
Compare
43fc7d4
to
594b159
Compare
c8a1e02
to
265e5f2
Compare
594b159
to
0a3b6c1
Compare
265e5f2
to
3042051
Compare
5925b99
to
13b61d5
Compare
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.
Reviewed 5 of 5 files at r1, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @ArniStarkware)
13b61d5
to
151ea3c
Compare
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)