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

Avoid liner: function not supported in this terminal #1955

Merged
merged 2 commits into from
Nov 28, 2019

Conversation

H34D
Copy link
Contributor

@H34D H34D commented Nov 28, 2019

Description

fix liner: function not supported in this terminal by giving docker some time to attach the tty

@jcortejoso jcortejoso self-requested a review November 28, 2019 09:29
Copy link
Contributor

@aaitor aaitor left a comment

Choose a reason for hiding this comment

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

:shipit:

@codecov
Copy link

codecov bot commented Nov 28, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1955   +/-   ##
======================================
  Coverage    74.2%   74.2%           
======================================
  Files         278     278           
  Lines        7675    7675           
  Branches      962     962           
======================================
  Hits         5695    5695           
  Misses       1863    1863           
  Partials      117     117
Flag Coverage Δ
#mobile 74.2% <ø> (ø) ⬆️

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 1a5745e...bbf593c. Read the comment docs.

@jcortejoso jcortejoso added the automerge Have PR merge automatically when checks pass label Nov 28, 2019
@celo-ci-bot-user celo-ci-bot-user merged commit e1d05a5 into celo-org:master Nov 28, 2019
@H34D H34D deleted the fix/liner branch November 28, 2019 13:44
aaronmgdr added a commit that referenced this pull request Dec 4, 2019
* master: (120 commits)
  Update docker image and instructions (#2000)
  DOCS correct typos (#1965)
  Add a space on payment request (#1935)
  expose signer through getValidator (#1997)
  Fix validator election migration bug with sorted list insertion (#1998)
  Deploy an oracle cronjob (#1814)
  Set default env in attestation service docker image as production (#1999)
  Attestation Bot POC (#1851)
  Only allow external RPCs for tx nodes (#1994)
  Specify web deps properly (#1950)
  Fix error calculating fees on currencies with high exchange rate (#1937)
  Small pre-stake-off CLI cleanup (#1953)
  Migrations: different CLabs groups gets different votes (#1960)
  [Wallet] Fix broken translation (decline, pay) (#1968)
  [Wallet]: Add Payments You've Requested notifications group & screen (#1902)
  [Wallet] Fix iOS bundle script failing with our monorepo setup (#1958)
  [NotificationService] Change exchange rate stored format (number timestamp and pair) (#1945)
  [Wallet] Fix premature hiding of syncing banner and increase some timeouts (#1957)
  Avoid `liner: function not supported in this terminal` (#1955)
  Support more human readable log output (#1929)
  ...
diminator pushed a commit to diminator/celo-monorepo that referenced this pull request Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants