-
-
Notifications
You must be signed in to change notification settings - Fork 159
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
Rename/remove a few policy params and archive some pre-TCJA reforms #2223
Rename/remove a few policy params and archive some pre-TCJA reforms #2223
Conversation
@martinholmer, these changes look good to me. @andersonfrailey, it looks like there's some potential for merge conflicts between your PR #2222 and @martinholmer's PR #2223. |
Reforms test fails on GitHub but passes on my computer. Will investigate this afternoon. |
Codecov Report
@@ Coverage Diff @@
## master #2223 +/- ##
======================================
Coverage 100% 100%
======================================
Files 12 12
Lines 2976 2981 +5
======================================
+ Hits 2976 2981 +5
Continue to review full report at Codecov.
|
This pull request resolves issue #2215.
Requesting review by @MattHJensen @codykallen @hdoupe @andersonfrailey @Peter-Metz