Skip to content

Conversation

@kraktus
Copy link
Contributor

@kraktus kraktus commented Nov 17, 2025

Hello,

A PR that show the changes made in https://github.com/kraktus/dataclass-csv2, for reference. I have no time nor desire to fix the merge issues, but feel free to do so.

We want `dataclass_csv2` on pypi but keep `dataclass_csv` to be a drop-in replacement.
taken from https://github.com/pypa/distutils/blob/ff11eed0c36b35bd68615a8ebf36763b7c8a6f28/distutils/util.py#L321

Fix dfurtado#52

Note that the utility of the use of this function has not been checked, a better refactor may be possible.
The whole code would probably need to be refactored but it should works for now

```
dataclass-csv2/dataclass_csv/dataclass_reader.py:149:76 - error: "key" is possibly unbound (reportPossiblyUnboundVariable)
```
@dfurtado
Copy link
Owner

Hey @kraktus thank you so much for putting this together!

Awesome additions! 👍🏽

The GitHub actions for testing the code using multiple Python versions was great! Exactly what I had in mind! The usage of uv will simplify a lot the dependency management and packaging, I didn't know this project so thanks a lot for introducing that.

I will make sure to add you to the list of contributors.

I will see what I can do to fix all the conflicts and get that merged asap.

Thank you!

Copy link
Owner

@dfurtado dfurtado left a comment

Choose a reason for hiding this comment

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

@kraktus Great work! I basically left all change you did in the fork with the exception of some minor changes.
If you are ok with the changes I can merge it right away.

strategy:
fail-fast: false
matrix:
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
Copy link
Owner

Choose a reason for hiding this comment

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

That's great! Exactly what I was going to do. 👍🏽

Copy link
Owner

Choose a reason for hiding this comment

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

Great you removed the stubs

{ url = "https://files.pythonhosted.org/packages/d1/d6/3965ed04c63042e047cb6a3e6ed1a63a35087b6a609aa3a15ed8ac56c221/colorama-0.4.6-py2.py3-none-any.whl", hash = "sha256:4f1d9991f5acc0ca119f9d443620b77f9d6b33703e51011c16baf57afb285fc6", size = 25335, upload-time = "2022-10-25T02:36:20.889Z" },
]

[[package]]
Copy link
Owner

Choose a reason for hiding this comment

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

I regenerated the lock file, and I noticed that the project has a dependency to itself. Can I remove this?

## Thanks

Thank you to all the amazing contributors who have supported this project over the years, with special thanks to [@kraktus](https://github.com/kraktus) for setting up GitHub Actions, improving automation for package creation, and making numerous code enhancements.

Copy link
Owner

Choose a reason for hiding this comment

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

Noted your work here, I hope it is ok.

@dfurtado dfurtado merged commit 6b103a2 into dfurtado:master Nov 20, 2025
18 checks passed
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