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

use logger from execution context in migrate #10

Closed
shahidhk opened this issue Jun 28, 2018 · 1 comment
Closed

use logger from execution context in migrate #10

shahidhk opened this issue Jun 28, 2018 · 1 comment
Labels
c/cli Related to CLI

Comments

@shahidhk
Copy link
Member

migrate library should take a logger object instead of calling logrus directly. If logger is not set, logrus.New() can be used.

@shahidhk shahidhk added the c/cli Related to CLI label Jul 6, 2018
@shahidhk shahidhk changed the title [cli] use logger from execution context in migrate use logger from execution context in migrate Jul 6, 2018
@shahidhk
Copy link
Member Author

shahidhk commented Jul 9, 2018

Fixed in 1bff452

@shahidhk shahidhk closed this as completed Jul 9, 2018
shahidhk pushed a commit to shahidhk/graphql-engine that referenced this issue Sep 13, 2018
beerose referenced this issue in beerose/graphql-engine May 16, 2020
* console: fix ~ react table first child width for event logs

* console: update ~ react table data positioned to center
soorajshankar pushed a commit to soorajshankar/graphql-engine that referenced this issue Jan 7, 2021
hgiasac pushed a commit to hgiasac/graphql-engine that referenced this issue Dec 19, 2023
GitOrigin-RevId: 9fef8708b5b820dab9527f010d50a84defbea985
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/cli Related to CLI
Projects
None yet
Development

No branches or pull requests

1 participant