Skip to content
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

Add feature for Serde #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add feature for Serde #8

wants to merge 2 commits into from

Conversation

lleyton
Copy link

@lleyton lleyton commented Jun 13, 2024

It's useful if you need to store the repo metadata into a different format. In our case, we're using it serialize it as JSON for Mellisearch.

@dralley
Copy link
Owner

dralley commented Jun 15, 2024

You're serializing the whole object? Those can get quite large with all of the file lists, which most people don't even care about

Copy link
Owner

@dralley dralley left a comment

Choose a reason for hiding this comment

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

@lleyton Could you describe a bit about your use case?

@lleyton
Copy link
Author

lleyton commented Jun 18, 2024

Yeah, in my use case, I'm serializing all of the packages within several repos to JSON in order to load them into a mellisearch index. My plan is to use this index for a web app which allows full-text package search (descriptions, names, author, etc..).

@dralley
Copy link
Owner

dralley commented Jun 18, 2024

Ok. You're sure you'll want everything and not a more restricted set of fields?

@lleyton
Copy link
Author

lleyton commented Jun 21, 2024

Hmm, it's likely that I'll want a more restricted set of fields when I'm further along. For prototyping though, I'm using all of them for now.

@dralley
Copy link
Owner

dralley commented Jun 21, 2024

I'm a little hesitant to merge this unless if it's not exactly what is needed. Would you mind if I waited to merge and release this until you are mostly done with your prototype?

@lleyton
Copy link
Author

lleyton commented Jun 21, 2024

Sounds good with me :)

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