search

Storia-AI / sage

Chat with any codebase with 2 commands
https://sage.storia.ai
Apache License 2.0
740 stars 56 forks source link

[FEATURE REQUEST] GitHub action to run existing unit tests on every PR #74

Open iuliaturc opened 1 week ago

iuliaturc commented 1 week ago

We have unit tests here. We should configure a GitHub automation to run them on every pull request.

CharlieFranzel commented 1 week ago

Im very new to open source, would you still want me to try and take this project on?

iuliaturc commented 1 week ago

Here are some instructions that might be helpful. Let me know if you'd like to take it :)

CharlieFranzel commented 1 week ago

Yes I'll be happy to take it on!

Pratik00019 commented 1 week ago

Hello @iuliaturc,

I have some experience working with Terraform scripts and I would love to contribute to this project. Could you please assign this issue to me?

Thank you!

iuliaturc commented 1 week ago

Hey @Pratik00019, thanks so much for offering to contribute!

I actually have a slightly harder one that is more worthy of your expertise. Would you be willing to take this one? https://github.com/Storia-AI/sage/issues/80

Pratik00019 commented 1 week ago

Hello @iuliaturc,

I currently don't have much experience with GitHub Actions. Is there any other issue that you think I can contribute to? I would love to help out in any way I can.

Thank you!

iuliaturc commented 1 week ago

@Pratik00019 you can choose any unassigned issue marked as hacktoberfest. If you need more help choosing or have any other questions, feel free to use our discord.

Spirizeon commented 6 days ago

Hi, I have implemented and worked with Github Actions in some projects, can I be assigned this issue as well?

iuliaturc commented 5 days ago

@CharlieFranzel Just checking in -- were you able to figure out how to address this issue? If you're struggling then I'm happy to assign you something else and leave this to @Spirizeon who seems to have more experience. Just let me know what you'd prefer @CharlieFranzel

manascb1344 commented 5 days ago

Hi, can I take up this issue??

manascb1344 commented 4 days ago

Hi, @iuliaturc can you review the PR I have made for this issue and inform me if anything more is required. Currently I cannot test the workflow as it needs the maintainers approval

iuliaturc commented 4 days ago

@manascb1344 Ah yes just did, apologies for the late response. Looks like it's failing with some dependencies errors, could you take another look please?