Skip to content
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

Removed benefit cap switch #1383

Merged
merged 1 commit into from
May 30, 2017
Merged

Conversation

hdoupe
Copy link
Collaborator

@hdoupe hdoupe commented May 26, 2017

Close #1373

@codecov-io
Copy link

codecov-io commented May 26, 2017

Codecov Report

Merging #1383 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1383   +/-   ##
=======================================
  Coverage   99.64%   99.64%           
=======================================
  Files          38       38           
  Lines        2824     2824           
=======================================
  Hits         2814     2814           
  Misses         10       10

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e24f8c...baf6bb0. Read the comment docs.

@econ02
Copy link
Collaborator

econ02 commented May 26, 2017

OK, the exemption is applied to the sum of the husband's and wife's labor
income. Note that the FICA tax is not part of the income tax, so a
taxpayer might have a lot of FICA and no income tax, so unless the
exemption is refundable it is not very valuable/expensive.

This isn't really an exemption on the FICA, it is a credit on the income
tax for FICA tax already paid.

dan

@feenberg is there any way you can clarify your comment on #1381 ?

@feenberg
Copy link
Contributor

feenberg commented May 26, 2017 via email

@MattHJensen
Copy link
Contributor

@hdoupe this looks good, and I am going to merge it. I am concerned about a source of potential user confusion, but will address it in a dedicated issue.

@MattHJensen MattHJensen merged commit 8d2372c into PSLmodels:master May 30, 2017
@hdoupe hdoupe deleted the renacci_reform branch May 30, 2017 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants