search

SeineAI / SeineSailor

Help developers navigating through the codebase smoothly
MIT License
4 stars 7 forks source link

update readme #1

Closed wenboown closed 5 months ago

wenboown commented 5 months ago

@SeineSailor review

wenboown commented 5 months ago

last run didn't work because gcr.io/distroless/python3-debian10 use python 3.7. langchain got an error.

I changed it to debian12 / python 3.11

@SeineSailor try again

wenboown commented 5 months ago

@SeineSailor test event payload

wenboown commented 5 months ago

with new code:

https://github.com/SeineAI/SeineSailor/blob/abf0714095889c8a7cc17d60fffc001d887fcbe5/.github/workflows/test_env.yml#L15-L21

and https://github.com/SeineAI/SeineSailor/blob/abf0714095889c8a7cc17d60fffc001d887fcbe5/.github/workflows/test_env.yml#L30-L36

@SeineSailor let's see if "Submit review" will trigger anything

the "Submit review" event triggered all the following calls at the same time:

Screenshot 2024-05-11 at 2 22 04 AM

https://github.com/SeineAI/SeineSailor/actions/runs/9041694890 https://github.com/SeineAI/SeineSailor/actions/runs/9041694893 https://github.com/SeineAI/SeineSailor/actions/runs/9041694895 https://github.com/SeineAI/SeineSailor/actions/runs/9041694896 https://github.com/SeineAI/SeineSailor/actions/runs/9041694897 https://github.com/SeineAI/SeineSailor/actions/runs/9041694899 https://github.com/SeineAI/SeineSailor/actions/runs/9041694900 https://github.com/SeineAI/SeineSailor/actions/runs/9041694903 https://github.com/SeineAI/SeineSailor/actions/runs/9041694907 https://github.com/SeineAI/SeineSailor/actions/runs/9041694908

maybe this is the problem that we didn't check the event type when checking comment.body: https://github.com/SeineAI/SeineSailor/blob/abf0714095889c8a7cc17d60fffc001d887fcbe5/.github/workflows/test_env.yml#L31

wenboown commented 5 months ago

@SeineSailor run

wenboown commented 5 months ago

@SeineSailor

consolidated test_env into llm-review in dc14589

wenboown commented 5 months ago

@SeineSailor added logging:

https://github.com/SeineAI/SeineSailor/blob/e82b8f3519b39376fcbc46b9a5a18ea4120a7228/app/main.py#L55

https://github.com/SeineAI/SeineSailor/blob/e82b8f3519b39376fcbc46b9a5a18ea4120a7228/app/context.py#L16-L17

also enabled runner and setup diagnosis logging: https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/enabling-debug-logging#enabling-runner-diagnostic-logging

wenboown commented 5 months ago

@SeineSailor Can you summarize all the testings we did so far in this pull request comments thread?

wenboown commented 5 months ago

@SeineSailor Can you summarize all the testings we did so far in this pull request comments thread?

triggered: https://github.com/SeineAI/SeineSailor/actions/runs/9085606147/job/24969337455

but: Skipped: this action only works on push events or pull_request