Skip to content

chore: make each package its own go module#4

Merged
andreynering merged 4 commits intomainfrom
charm-607-migrate-each-provider-package-into-a-go-module
Sep 3, 2025
Merged

chore: make each package its own go module#4
andreynering merged 4 commits intomainfrom
charm-607-migrate-each-provider-package-into-a-go-module

Conversation

@andreynering
Copy link
Copy Markdown
Member

@andreynering andreynering commented Sep 2, 2025

  • ./ai (previously root /)
  • ./examples
  • ./providers/anthropic
  • ./providers/openai

TODO:

  • Adjust CI to run tests and linters for each module
  • Think a little if this will work "for real".
    • We are currently using replace directives, which work great locally, but probably won't for importers.
    • We'll need to tag the same version for each package when releasing.
    • How to manage that, though? We have an inter-dependency here. Providers will always import ./ai, but can we tag ./ai first and update all the other packages in ./providers later? Will we need to do it in two steps (commits)?

@andreynering andreynering self-assigned this Sep 2, 2025
* `./ai` (previously root `/`)
* `./examples`
* `./providers/anthropic`
* `./providers/openai`
@andreynering andreynering force-pushed the charm-607-migrate-each-provider-package-into-a-go-module branch from 4682807 to 1a8c0fc Compare September 2, 2025 20:57
Copied generate script from `charmbracelet/x`.
@andreynering andreynering force-pushed the charm-607-migrate-each-provider-package-into-a-go-module branch from 1a8c0fc to 125ce75 Compare September 2, 2025 20:59
@kujtimiihoxha
Copy link
Copy Markdown
Contributor

@andreynering I installed it in my branch and it seems to be working
Screenshot 2025-09-03 at 11 31 25
it compiles as expected

@andreynering andreynering marked this pull request as ready for review September 3, 2025 13:10
@andreynering
Copy link
Copy Markdown
Member Author

Looks like this will work as expected, just need a two step release, maybe.

I'll merge so we can continue working on it, but we can do adjustments before the release and even test on a test repo, maybe.

Crush conversartion screenshots

Screenshot 2025-09-03 at 10 10 13 Screenshot 2025-09-03 at 10 10 31

@andreynering andreynering merged commit 5337bf2 into main Sep 3, 2025
28 checks passed
@andreynering andreynering deleted the charm-607-migrate-each-provider-package-into-a-go-module branch September 3, 2025 13:14
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