search

DevChatter / ChatterBot

Highly modular Twitch chat bot.
MIT License
10 stars 5 forks source link

#9 GitHub actions build pipeline - attempt 1 of ?? (yay for YAML) #32

Closed SimonGeering closed 4 years ago

SimonGeering commented 4 years ago

Step 1 of #9 providing an initial CI/CD build pipeline;

This is just the simple PR workflow that will compile the solution and run all the tests in the test project.

Since we may want to filter tests by category later, so ill add annotations to the existing unit tests for that, as it will also give me something to PR into my own main branch to test the triggers.

This also fixes #33 with the resolution of a null reference bug.

SimonGeering commented 4 years ago

@benrick, after this PR is merged to main would you also be able to get the dev branch cut from main please? Not only would that allow the other trigger for this pipeline to be tested, but it would also be best if I were to make future PR branches off of dev as a step towards implementing the final structure in #23

SimonGeering commented 4 years ago

Obviously you will need to set-up the branch protection rules once it's in on your side. If I recall correctly the build has to run once before you can enforce it as a constraint on a PR, but you should still be able to enable Require status checks to pass before merging