Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Bump clippy & Fix warnings #2109

Merged
merged 4 commits into from
Sep 16, 2016
Merged

Bump clippy & Fix warnings #2109

merged 4 commits into from
Sep 16, 2016

Conversation

tomusdrw
Copy link
Collaborator

No description provided.

@tomusdrw tomusdrw added the A0-pleasereview 🤓 Pull request needs code review. label Sep 16, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 85.242% when pulling 35c9039 on bump-clippy into 851b639 on master.

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 16, 2016
@arkpar arkpar merged commit 7f7e748 into master Sep 16, 2016
@tomusdrw tomusdrw deleted the bump-clippy branch September 17, 2016 13:35
jacogr added a commit that referenced this pull request Sep 19, 2016
* js: (59 commits)
  use standard modal title functionality
  fix modal title size where no steps are available
  get adding addresses working again (context migration)
  Bump clippy & Fix warnings (#2109)
  Added docs
  fix merge conflict
  check for existence of deprecated ethash file before attempting delete (#2103)
  fix grumble
  check response.ok, throw Error
  array for classlist
  .map -> .forEach
  fix wording
  rename fundAccount to shapeshift
  use shapeshift ison & name
  fix typo
  use only one image for tokens, scale it (maps what is available when pulled form IMG meta in TokenReg)
  Import summary
  unknown image when none found for a token
  don't show tokens.value === 0 in transfer
  cater for the cases where the token doesn't have an image
  ...

# Conflicts:
#	js/src/modals/Transfer/Details/details.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants