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

Move State to data package #835

Merged
merged 1 commit into from
Jan 30, 2016
Merged

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Jan 30, 2016

This is a partial partial fix for #785. So far I've moved state but not free. There are several different types within the free package, so it might warrant its own package. However, State/StateT was just sitting in the state package by itself, so I've gone ahead and moved it (so it now parallels Writer/WriterT).

@adelbertc
Copy link
Contributor

👍 once we have green light

@codecov-io
Copy link

Current coverage is 89.30%

Merging #835 into master will not affect coverage as of 3fc1cfe

@@            master   #835   diff @@
=====================================
  Files          168    168       
  Stmts         2319   2319       
  Branches        75     75       
  Methods          0      0       
=====================================
  Hit           2071   2071       
  Partial          0      0       
  Missed         248    248       

Review entire Coverage Diff as of 3fc1cfe

Powered by Codecov. Updated on successful CI builds.

@fthomas
Copy link
Member

fthomas commented Jan 30, 2016

👍

fthomas added a commit that referenced this pull request Jan 30, 2016
@fthomas fthomas merged commit 4f0e90e into typelevel:master Jan 30, 2016
@ceedubs ceedubs deleted the state-data-pkg branch February 2, 2016 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants