thmsmlr / instructor_ex

Structured outputs for LLMs in Elixir
https://hexdocs.pm/instructor
430 stars 48 forks source link

Add first GitHub CI setup #28

Closed thbar closed 3 months ago

thbar commented 4 months ago

Taking a stab at:

I looked for fresh inspiration and decided to try out this:

Connected tests are not currently running (which is a good start), due to how the tests are configured:

https://github.com/thmsmlr/instructor_ex/blob/617ee4c813f1189abde76130c7a636fe2a1a0c99/test/test_helper.exs#L3-L4

The configured matrix tests:

  1. The most recent Elixir/OTP combination, with linting activated
  2. An older combination for compatibility check

Creating the PR so that we can try things out.

TODOs

thbar commented 4 months ago

@thmsmlr we'll need to figure out how to test this before merging the PR. Hit me up if you see some options on your side!

EDIT: check out https://stackoverflow.com/questions/67707244/how-to-test-github-workflow-without-merging-into-master-main-branch and such :smile:

thmsmlr commented 3 months ago

Thanks for the PR @thbar , i couldn't figure out to test on this branch since it was external? idk github permissions are confusing. Debugged a few things on a different branch and got it merged into main. Feel free to submit changes to it and we can test more normally

thbar commented 3 months ago

@thmsmlr no problem, we got the ball rolling this way :smile: thanks for accepting the PR & tweaking it!