-
Notifications
You must be signed in to change notification settings - Fork 188
2021, 2022, 2024 State Sales Tax #5591
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?
2021, 2022, 2024 State Sales Tax #5591
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5591 +/- ##
==========================================
- Coverage 99.10% 99.07% -0.03%
==========================================
Files 2687 2695 +8
Lines 38934 38988 +54
Branches 170 171 +1
==========================================
+ Hits 38585 38629 +44
- Misses 315 326 +11
+ Partials 34 33 -1 ☔ View full report in Codecov by Sentry. |
Lets add a changelog entry and and test for 2021 |
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.
also please:
- fix the metadata for income_bracket.yaml
- cite other years' sources in all three yaml files
- try to find statutory or regulatory backing for this program; if not available, note that in a comment
- add the script you used to generate the yaml content
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.
add references for other years
@@ -8,6 +8,32 @@ metadata: | |||
href: https://www.irs.gov/instructions/i1040sca#en_US_2024_publink1000131499 | |||
|
|||
values: | |||
2021-01-01: |
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.
don't need new yearly entries if it hasn't changed
just checking in on this |
Fixes #5589