Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update develop.md #14025

Merged
merged 2 commits into from
Feb 24, 2022
Merged

Update develop.md #14025

merged 2 commits into from
Feb 24, 2022

Conversation

ogunsoladebayo
Copy link
Contributor

Proposed changes

  • provided more clarity to the structure of the mongo database connection string used as CONNECTIONSTRING environment variable to run the docker image in the sample node.js project.

- provided more clarity to the structure of the mongo database connection string used as CONNECTIONSTRING environment variable to run the docker image in the sample node.js project.
@netlify
Copy link

netlify bot commented Dec 21, 2021

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: 9bfdf64

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/61c5acb80d5f9800074a8bce

😎 Browse the preview: https://deploy-preview-14025--docsdocker.netlify.app

Copy link
Member

@usha-mandya usha-mandya left a comment

Choose a reason for hiding this comment

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

LGTM

@usha-mandya usha-mandya reopened this Feb 24, 2022
@usha-mandya
Copy link
Member

Close and reopen the PR to trigger CI checks

@usha-mandya usha-mandya merged commit f5ed385 into docker:master Feb 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants