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

Fix return #1825

Merged
merged 3 commits into from
Nov 21, 2019
Merged

Fix return #1825

merged 3 commits into from
Nov 21, 2019

Conversation

H34D
Copy link
Contributor

@H34D H34D commented Nov 21, 2019

Description

return at least true, when the function is returning bool

@H34D H34D requested review from asaj and m-chrzan as code owners November 21, 2019 08:01
@codecov
Copy link

codecov bot commented Nov 21, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1825   +/-   ##
======================================
  Coverage    74.2%   74.2%           
======================================
  Files         278     278           
  Lines        7653    7653           
  Branches      956     956           
======================================
  Hits         5679    5679           
  Misses       1857    1857           
  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 d7a5c45...dd3c898. Read the comment docs.

@H34D
Copy link
Contributor Author

H34D commented Nov 21, 2019

@m-chrzan i am not authorised to merge

@m-chrzan m-chrzan merged commit 85c7009 into celo-org:master Nov 21, 2019
@H34D H34D deleted the fix/return branch November 21, 2019 15:18
aaronmgdr added a commit that referenced this pull request Nov 26, 2019
* master: (61 commits)
  [Wallet] Handle `/v/<code` deep links for phone verifications (#1776)
  Patch tslint auto fix for macOS Catalina until tslint#6.x is out (#1802)
  Fix typos (#1855)
  Update documentation sidebar (#1861)
  [Wallet] Update forno dev documentation (#1818)
  [Docs] Correct typo
  Validator UX docs for baklava network (#1688) (#1849)
  Various improvements to the CLI, allow voters to revote for a group (#1840)
  Serialize to hex string (#1848)
  Update faucet to pull from the reserve when possible (#1844)
  Fix elect validators migration, deploy integration (#1847)
  Configurable genesis balances (#1838)
  Deploy new version of Celostats and minnor change from celo-blockchain (#1714)
  Make default node URL consistent in celocli (#1805)
  Onboarding feedback (#1811)
  return at least true, when the function is returning bool (#1825)
  Minor Contractkit changes (#1819)
  Point end-to-end tests back to master (#1824)
  Update migration config to correct protocol parameters (#1822)
  Update with new istanbul lookback window size flag (#1820)
  ...

# Conflicts:
#	yarn.lock
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants