Skip to content

Conversation

@Mshuning
Copy link
Collaborator

@Mshuning Mshuning commented Sep 6, 2024

Fixes #4817

@Mshuning Mshuning changed the title LA County LIFE transit discount Refactor the regular_tax_before_credits variable and add unit tests Sep 6, 2024
@Mshuning Mshuning changed the title Refactor the regular_tax_before_credits variable and add unit tests Refactor the regular_tax_before_credits variable and add unit tests Sep 6, 2024
Copy link
Collaborator

@PavelMakarchuk PavelMakarchuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a start, can you add the relevant tax form lines, especially for the ambiguous variables names
e.g.

# Line 17
dwks19 = ... 

@Mshuning
Copy link
Collaborator Author

Mshuning commented Sep 19, 2024

2017 Instructions for Schedule D
image
2023 Instructions for Schedule D
image
@PavelMakarchuk The differences between the 2017 form and the 2023 form are from step 19 in 2017 to steps 19-21 in 2023.

@Mshuning
Copy link
Collaborator Author

@nikhilwoodruff nikhilwoodruff force-pushed the master branch 2 times, most recently from 7926ec9 to d2543dc Compare July 12, 2025 15:13
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.

Refactor the regular_tax_before_credits variable and add unit tests

2 participants