programmer-fa icon indicating copy to clipboard operation
programmer-fa copied to clipboard

Setup GitHub Actions for CD

Open amirhoseinsalimi opened this issue 2 years ago • 10 comments

amirhoseinsalimi avatar Apr 29 '22 17:04 amirhoseinsalimi

I would be happy to contribute. Can you provide me with more information about it?

shywn-mrk avatar Apr 29 '22 17:04 shywn-mrk

Wow. I've just opened this. How did you find this repo so quickly?

I just want to setup auto-deployment for this project, so whenever a commit is made to the master branch, it gets deployed on the production server.

I think one of the key points here is going to be tests, so I should write more tests to be more confident.

Now, what is your opinion about this? What should we do before making this happen?

amirhoseinsalimi avatar Apr 29 '22 17:04 amirhoseinsalimi

Having tests such as unit tests can be checked before deploying to production. A simple CI/CD for this project can be written in a short amount of time but I need more details. Writing tests, however, is not related to the pipelines. Our concerns should be separated.

shywn-mrk avatar May 01 '22 08:05 shywn-mrk

I agree. I don't have much knowledge to create an action script for GitHub actions. Could you write one? (Time is not a concern, just asking)

amirhoseinsalimi avatar May 01 '22 10:05 amirhoseinsalimi

I forgot something. What kind of information do you need? Getting started with the project is already documented in the README. What else do you need?

amirhoseinsalimi avatar May 01 '22 10:05 amirhoseinsalimi

Where should we exactly deploy? The runners should be configured.

shywn-mrk avatar May 01 '22 10:05 shywn-mrk

On my personal server. How can we co-operate?

amirhoseinsalimi avatar May 01 '22 11:05 amirhoseinsalimi

Currently, I'm on vacation but we can have a meeting. I will contact you next week.

shywn-mrk avatar May 02 '22 09:05 shywn-mrk

I would be happy to contribute if this issue still exists.

MiKDev avatar May 18 '22 13:05 MiKDev

Thanks. Going to merge.

amirhoseinsalimi avatar May 18 '22 14:05 amirhoseinsalimi