Skip to content

Conversation

@Andarist
Copy link
Contributor

no actual change now, just preventive measure in case u add some deps in the future

@codecov
Copy link

codecov bot commented Apr 12, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #222   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           8      8           
  Lines         169    169           
  Branches       57     57           
=====================================
  Hits          169    169

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 cf00e65...eb5d980. Read the comment docs.

@erikras
Copy link
Member

erikras commented Apr 12, 2018

Interesting. By "preventive measure", what is it preventing? Me adding a dep but forgetting to put it into the rollup.config.js?

@Andarist
Copy link
Contributor Author

Interesting. By "preventive measure", what is it preventing? Me adding a dep but forgetting to put it into the rollup.config.js?

Exactly

@erikras erikras merged commit ffabee5 into final-form:master Apr 12, 2018
@erikras
Copy link
Member

erikras commented Apr 12, 2018

Published in v3.3.0.

@lock
Copy link

lock bot commented May 31, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2018
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