Skip to content

Conversation

@zhangjx
Copy link
Contributor

@zhangjx zhangjx commented May 11, 2017

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change
Environment

Node: v6.9.0
egg-sequelize: 2.1.3

Description

When use npm run migrate:up always error, because the config file is covered by the production environment. See #895

@mention-bot
Copy link

@zhangjx, thanks for your PR! By analyzing the history of the files in this pull request, we identified @huacnlee to be a potential reviewer.

@codecov
Copy link

codecov bot commented May 11, 2017

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #14   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           5      5           
  Lines          36     36           
=====================================
  Hits           36     36

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 773c750...89c0951. Read the comment docs.

@popomore popomore merged commit f64ca5f into eggjs:master May 11, 2017
@popomore
Copy link
Member

  • egg-sequelize@2.1.4

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.

3 participants