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

quick ixmonad and examples #55

Merged
merged 4 commits into from
Jan 5, 2024
Merged

Conversation

jldodds
Copy link
Contributor

@jldodds jldodds commented Feb 1, 2019

No description provided.

@gmalecha
Copy link
Collaborator

gmalecha commented Feb 5, 2019

I meant to get back to you earlier about this. I think developments like this are great, but I've been wondering if they should be split into separate libraries. What do you think of making coq-el-indexed (we can put it in this organization) and release it as a separate package?

@jldodds
Copy link
Contributor Author

jldodds commented Feb 5, 2019

That makes sense, although this would be a pretty small package, the only instances I'm aware of are state and cont.

It's completely reasonable to close this until someone has time/interest to actually get a complete implementation done.

@gmalecha
Copy link
Collaborator

gmalecha commented Feb 5, 2019

It wouldn't be as small as some node.js packages. :-)

What is left for a "complete implementation"?

@jldodds
Copy link
Contributor Author

jldodds commented Feb 5, 2019

It's certainly usable as is, so maybe nothing needs to be added

@liyishuai
Copy link
Member

@gmalecha Should we merge this PR?

@gmalecha
Copy link
Collaborator

I don't have any real technical objections to merging this, but I think the question is whether ExtLib should grow and/or shrink. Does it make sense to release separate packages with add-on functionality like indexed monads? What needs to be in the core?

@liyishuai
Copy link
Member

It'll make sense to release a separate package if no future code in ExtLib would depend on such add-ons.
I'd prefer having ExtLib as a growing and shrinking library where the community can contribute functionalities they find generically useful and should not redefine everywhere. When some code were adopted by Coq standard library, we should remove them from here.
Some functionalities could come in different styles (e.g. monad notations), and the Coq team might have difficulty deciding which one to include in standard library. In this case, ExtLib can host one or many of them, allowing users to choose.

@github-actions github-actions bot added the Stale label Sep 22, 2023
@liyishuai liyishuai removed the Stale label Jan 4, 2024
@liyishuai liyishuai enabled auto-merge (squash) January 4, 2024 07:36
@liyishuai liyishuai merged commit c315bb0 into coq-community:master Jan 5, 2024
10 checks passed
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.

3 participants