Skip to content

Conversation

@ctmartinez1992
Copy link
Contributor

I wasn't sure where to place this information. I placed it in the testing section, but it seems like it could make sense in the stack and tools section as well.

I made a new page just for this. Right now it only explains how to run Django tests in Github Actions. It makes a lot of assumptions like using poetry and Postgres, but I think that's ok because people can easily infer what may be necessary just by having an example.

I made no mention to the caching possibilities of Github Actions as it can create confusion.

@ctmartinez1992 ctmartinez1992 self-assigned this Mar 3, 2023
@ctmartinez1992 ctmartinez1992 added the documentation Improvements or additions to documentation label Mar 3, 2023
JdFSilva
JdFSilva previously approved these changes Mar 9, 2023
@ctmartinez1992
Copy link
Contributor Author

I added a github action to run the tests and only deploy if those tests run successfully. Also, the test job always runs on each push, but the deploy job only runs if it's the master branch that received the push.

There's a small issue though: the test job took almost 18 minutes to run and most of its time was spent in the install poetry dependencies step. Is there any reason for this to take so long? I also noticed that when I installed the dependencies locally, it also took a very long time to complete. @JdFSilva @kenvontucky

JdFSilva
JdFSilva previously approved these changes Mar 17, 2023
goncaloadolfo
goncaloadolfo previously approved these changes Apr 12, 2023
@ctmartinez1992 ctmartinez1992 dismissed stale reviews from goncaloadolfo and JdFSilva via 90351f3 April 26, 2023 09:09
ghost
ghost previously approved these changes Apr 26, 2023
goncaloadolfo
goncaloadolfo previously approved these changes Apr 27, 2023
@ctmartinez1992 ctmartinez1992 dismissed stale reviews from goncaloadolfo and ghost via e52258a April 27, 2023 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants