-
Notifications
You must be signed in to change notification settings - Fork 188
Puerto Rico Refundable CTC - form 1040 computation #5925
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
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5925 +/- ##
=======================================
Coverage 99.10% 99.10%
=======================================
Files 2817 2819 +2
Lines 40742 40766 +24
Branches 190 190
=======================================
+ Hits 40379 40403 +24
Misses 328 328
Partials 35 35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 after minor format and disclaimer
...ngine_us/variables/gov/territories/pr/tax/income/credits/refundable_ctc/pr_refundable_ctc.py
Outdated
Show resolved
Hide resolved
...ov/territories/pr/tax/income/credits/refundable_ctc/pr_refundable_ctc_social_security_tax.py
Outdated
Show resolved
Hide resolved
# This provision is part of the federal CTC legal code | ||
# will will merge the logic with the federal CTC once the puerto income tax structure is completed | ||
def formula(tax_unit, period, parameters): | ||
# line 24 |
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.
Could you add the tax form to the reference section in all variables where we use the line comments?
@@ -3,7 +3,7 @@ | |||
|
|||
class pr_gross_income(Variable): |
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.
class pr_gross_income(Variable): | |
class pr_gross_income_person(Variable): |
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.
assuming that we will need a taxunit level gross income concept down the line
Fixes #5923