Skip to content
This repository was archived by the owner on Apr 7, 2021. It is now read-only.

👷 added mongo to github actions ci & npm test #5

Merged
merged 1 commit into from
Feb 25, 2020
Merged

Conversation

bmiddha
Copy link
Member

@bmiddha bmiddha commented Feb 24, 2020

Updated API GitHub Actions to create and run a mongodb container and run npm run test.

@bmiddha bmiddha requested a review from alexchomiak February 24, 2020 19:33
@bmiddha bmiddha self-assigned this Feb 24, 2020
Copy link
Member

@alexchomiak alexchomiak left a comment

Choose a reason for hiding this comment

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

LGTM, would this need changes if we eventually switched to atlas?

@bmiddha
Copy link
Member Author

bmiddha commented Feb 25, 2020

@alexchomiak, Not really. This is useful if we have test scripts which need mongodb. The ones included in the boilerplate do need it. The tests were probably stripped during the skeleton PR #2. If we need it in the future (we probably will), now we have it.
We should only be using a cloud instance for production.
I have a PR open for that to fixit in the boilerplate. microsoft/TypeScript-Node-Starter#268 (adds azure deploy and mongodb to the ci test)

@bmiddha bmiddha merged commit d397251 into master Feb 25, 2020
@bmiddha bmiddha deleted the ci-patch branch February 25, 2020 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants