chore(ci): basic CI set up#28
Conversation
|
CI checks failing right now, I presume that's expected since frontend and backend aren't set up to these naming conventions right now? |
|
I do not think that we need two different CI's, one will suffice, and we will be able to set up job paths as we continue through this project. We do not have anything to build at the moment, so I don't think that we should be adding jobs for our CI to build just yet. I left a suggestion on managing frontend vs backend tasks, and these can both be built within the same CI file. Moving to having just one CI will make the Pull Request process faster as well, as it will be one less environment to build (jest will be built twice in your configuration). |
|
@matt-dahlgren Thanks for the feedback! Updated the CI accordingly, still going to retain this as a draft until we rename our frontend and backend branches. |
|
Closed as updated in #44 |
Proposed Changes
Closes #3
Basic setup for CI pipeline for jest tests. Requires #36.
Type of Change
(Write an
Xor a brief description next to the type or types that best describe your changes.)Checklist
(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the
[ ]into a[x]in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)Before opening your pull request:
After opening your pull request:
Questions and Comments
@matt-dahlgren This is my first time setting up CI, could you take a look and see if there are any issues?