Skip to content
This repository was archived by the owner on May 17, 2019. It is now read-only.

Conversation

@hugoe29
Copy link
Contributor

@hugoe29 hugoe29 commented Feb 6, 2019

No description provided.

@codecov
Copy link

codecov bot commented Feb 6, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@82d878b). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #181   +/-   ##
=========================================
  Coverage          ?   93.33%           
=========================================
  Files             ?        2           
  Lines             ?       30           
  Branches          ?        5           
=========================================
  Hits              ?       28           
  Misses            ?        0           
  Partials          ?        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 82d878b...3b7ded3. Read the comment docs.

@lhorie lhorie added the ci label Mar 4, 2019
@hugoe29 hugoe29 changed the title Update dependency fusion-rpc-redux to ^1.4.0 Import and export ActionType from dependency fusion-rpc-redux ^1.4.0 Mar 4, 2019
@lhorie lhorie added the ci label Mar 4, 2019
src/index.js Outdated
ResponseError,
} from 'fusion-plugin-rpc';
import {createRPCReducer} from 'fusion-rpc-redux';
import {createRPCReducer, ActionType} from 'fusion-rpc-redux';
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be import type {ActionType}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@lhorie lhorie added the ci label Mar 4, 2019
@lhorie
Copy link
Contributor

lhorie commented Mar 4, 2019

Looks good to me! Thanks @hugoe29 !

@lhorie
Copy link
Contributor

lhorie commented Mar 4, 2019

Oh, the export also needed to be changed. I went ahead and made the adjustment

@hugoe29
Copy link
Contributor Author

hugoe29 commented Mar 4, 2019

Thanks @lhorie
It seems it is just waiting for the reviewer approval.

@lhorie lhorie merged commit 7f7957f into fusionjs:master Mar 5, 2019
@old-fusion-bot
Copy link

old-fusion-bot bot commented Mar 5, 2019

Triggered Fusion.js build verification: https://buildkite.com/uberopensource/fusion-release-verification/builds/1655

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants