Skip to content

Conversation

@Mr-Sunglasses
Copy link
Contributor

Issue: #3

  • run unit tests when PRs created

Copy link
Member

@Mariatta Mariatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for starting this! I suggest using the make test and make migrate commands.

@Mr-Sunglasses
Copy link
Contributor Author

Thank you for starting this! I suggest using the make test and make migrate commands.

Thanks @Mariatta I've done the changes, think the tests action is failing because as these are no tests and it can't able to generate the report, apart from that I think we are good to go.

@Mr-Sunglasses
Copy link
Contributor Author

requesting @Mariatta for a review

@Mr-Sunglasses
Copy link
Contributor Author

@Mariatta I think this PR is good to go, we can merge it.

@netlify
Copy link

netlify bot commented May 3, 2025

Deploy Preview for pyladiescon-portal-docs ready!

Name Link
🔨 Latest commit cabd7d2
🔍 Latest deploy log https://app.netlify.com/sites/pyladiescon-portal-docs/deploys/68194dc908268c0008ea137a
😎 Deploy Preview https://deploy-preview-78--pyladiescon-portal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Mariatta
Copy link
Member

Mariatta commented May 5, 2025

Thanks for the PR, but actually the tests were already done as part of the CI workflow.

@Mariatta Mariatta closed this May 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants