Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove msgspec dependency #630

Merged
merged 1 commit into from
Nov 28, 2024
Merged

Remove msgspec dependency #630

merged 1 commit into from
Nov 28, 2024

Conversation

rumpelsepp
Copy link
Member

@rumpelsepp rumpelsepp commented Nov 28, 2024 •
edited
Loading

  • feat: Use uv instead of poetry
  • chore: Unvendor pydantic_argparse and wire up uv correctly
  • chore: Enable ruff for pydantic_argparse
  • chore: Enable mypy for pydantic_argparse
  • fix: Remove msgspec dependency

Depends on #629

@rumpelsepp
fix: Remove msgspec dependency
f196c43 
The builtin json module works just fine. It is a bit slower for large
files, but this will be fixed with the upcoming rust based logfile
parser.
@rumpelsepp rumpelsepp merged commit 06a8b6b into master Nov 28, 2024
@rumpelsepp rumpelsepp deleted the msgspec branch November 28, 2024 14:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@fkglr fkglr Awaiting requested review from fkglr

@peckto peckto Awaiting requested review from peckto peckto is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rumpelsepp