Skip to content

Comments

More changes required to eliminate pylint warnings#402

Merged
martinholmer merged 6 commits intomasterfrom
more-style-changes
Feb 18, 2026
Merged

More changes required to eliminate pylint warnings#402
martinholmer merged 6 commits intomasterfrom
more-style-changes

Conversation

@martinholmer
Copy link
Collaborator

@martinholmer martinholmer commented Feb 17, 2026

Changes are cosmetic rather than substantive.

While looking at pylint warnings genratred by the tmd/utils/is_tax_filer.py code, I discovered that the is_tax_filer function is not called anywhere in the TMD repo code. So, I added at comment indicating that fact (rather than removing the module).

@martinholmer martinholmer merged commit 3e773a3 into master Feb 18, 2026
1 check passed
@martinholmer martinholmer deleted the more-style-changes branch February 18, 2026 13:57
@donboyd5
Copy link
Collaborator

donboyd5 commented Feb 18, 2026

Thank you, @martinholmer. I believe it will be potentially useful, sometime in the future, to examine that module carefully, update it as needed, and implement it. It could enable us to make modest improvements to how we target data. Doing that might be far in the future, but it's worth retaining the module, as you have done.

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.

2 participants