Skip to content

golang support#48

Open
classload-ux wants to merge 2 commits intorun-llama:mainfrom
classload-ux:golang-support
Open

golang support#48
classload-ux wants to merge 2 commits intorun-llama:mainfrom
classload-ux:golang-support

Conversation

@classload-ux
Copy link
Copy Markdown

No description provided.

@logan-markewich
Copy link
Copy Markdown
Contributor

I appreciate the contribution!

This doesn't really look like its in a place to merge though? I see a lot of console logs and hardcoded paths that don't quite make sense in a public package?

I'd love to add golang support, but tbh I know zero about go 😅 I think ideally later on we rewrite from typescript to rust and provide bindings to languages like golang. Not sure I want to maintain this contribution just yet

@classload-ux
Copy link
Copy Markdown
Author

Thanks for your feedback!

The hardcoded paths and console logs I added are only for testing purposes and won’t affect normal usage or the public package at all. I can clean them up before a formal merge if needed.

I totally understand you’re not familiar with Go and have future plans to rewrite the core in Rust and provide bindings for different languages. I really appreciate you considering Go support. If you’d like, I can help maintain this Go implementation later or adjust the code to better fit the project structure.

Thanks again for your hard work on this project!

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