Skip to content

Conversation

@L8D
Copy link

@L8D L8D commented Mar 26, 2015

No description provided.

@L8D
Copy link
Author

L8D commented Mar 26, 2015

it's missing the Store and Comonad stuff though, but it doesn't need that to be function.

Copy link
Member

Choose a reason for hiding this comment

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

Idris has confusing rules around this. Since you've marked one identifier as private you have to mark the ones which are meant to be public:

idris-lang/Idris-dev#1879

Sadly, it's all or nothing.

Copy link
Author

Choose a reason for hiding this comment

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

But doesn't %access public help with this?

Copy link
Member

Choose a reason for hiding this comment

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

Oh... Forgot about that, I think that solves the problem.

@L8D
Copy link
Author

L8D commented Mar 26, 2015

I'll add some doc strings...

@L8D
Copy link
Author

L8D commented Mar 26, 2015

I don't know if it would be practical to add more parameters to Lens for proofs that it follows the laws.

@puffnfresh
Copy link
Member

VerifiedLens would be awesome.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants