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

Migrate to new fusion-tokens api #70

Merged
merged 1 commit into from
Feb 2, 2018
Merged

Conversation

ganemone
Copy link
Contributor

@ganemone ganemone commented Feb 2, 2018

Fixes #69

@codecov
Copy link

codecov bot commented Feb 2, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   86.95%   86.95%           
=======================================
  Files           4        4           
  Lines         115      115           
  Branches       20       20           
=======================================
  Hits          100      100           
  Misses          8        8           
  Partials        7        7
Impacted Files Coverage Δ
src/index.js 100% <ø> (ø) ⬆️

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 8975c42...cf763cb. Read the comment docs.

@@ -36,17 +36,17 @@
"eslint-plugin-prettier": "2.5.0",
"eslint-plugin-react": "7.6.1",
"flow-bin": "0.64.0",
"fusion-core": "0.3.0",
"fusion-core": "^0.3.1",
Copy link
Member

@AlexMSmithCA AlexMSmithCA Feb 2, 2018

Choose a reason for hiding this comment

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

Are we moving towards relaxing semver requirements for all internal packages? Also should the lower-bound be v0.3.3 for better type safety?

@ganemone
Copy link
Contributor Author

ganemone commented Feb 2, 2018

!merge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants