Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Docker/setup #227

Merged
merged 1 commit into from
Nov 15, 2019
Merged

Docker/setup #227

merged 1 commit into from
Nov 15, 2019

Conversation

jannyHou
Copy link
Contributor

@jannyHou jannyHou commented Nov 12, 2019

Description

Status: code works but need cleanup.

Setup a couchdb 3 docker image for travis testing (without running tests in this repo).
My test app runs ping tests successfully on travis and get the exported env configs, see jannyHou/test-couchdb-travis#1

This script is created to unblock loopbackio/loopback-next#3437

Related issues

  • connect to <link_to_referenced_issue>

Checklist

  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style
    guide

docker.setup.js Outdated
// we don't pass any node flags, so we can call _mocha instead the wrapper
const mochaBin = require.resolve('mocha/bin/_mocha');

process.env.CLOUDANT_DATABASE = 'test-db';
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we change this to testdb? Most of our default db names are testdb 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@agnes512 sure whatever name makes more sense to lb-next tests~

docker.setup.js Outdated
const fs = require('fs-extra');

// we don't pass any node flags, so we can call _mocha instead the wrapper
const mochaBin = require.resolve('mocha/bin/_mocha');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@agnes512 I will remove this broken line :p

@jannyHou jannyHou force-pushed the docker/setup branch 2 times, most recently from e7312cc to a91ff11 Compare November 15, 2019 21:42
@jannyHou jannyHou changed the title [WIP]Docker/setup Docker/setup Nov 15, 2019
Copy link
Contributor

@agnes512 agnes512 left a comment

Choose a reason for hiding this comment

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

👍 🚢

@jannyHou jannyHou force-pushed the docker/setup branch 2 times, most recently from 0faa2c9 to a3a7e25 Compare November 15, 2019 22:16
@jannyHou jannyHou merged commit fae7941 into master Nov 15, 2019
@delete-merged-branch delete-merged-branch bot deleted the docker/setup branch November 15, 2019 22:29
dhmlau added a commit that referenced this pull request Jan 27, 2020
 * chore: update copyright year (Agnes Lin)
 * replace couchdb3 docker img with a stable version (Agnes Lin)
 * docs: add partition document (#232) (Janny)
 * fixup!: fix the dependency version (#233) (Janny)
 * feat: query with partition field (#230) (Janny)
 * chore: improve issue and PR templates (Nora)
 * feat: add partitioned find (#229) (Janny)
 * add docker setup script (#227) (Janny)
 * feat: partitioned index (#225) (Janny)
 * fix CODEOWNERS file (Diana Lau)
 * update docker image (#224) (Janny)
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