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

Don't claim cubical compiles with the latest agda development version #717

Merged
merged 9 commits into from
Feb 11, 2022

Conversation

felixwellen
Copy link
Collaborator

The readme claims, that the library compiles with the latest version of agda. That is currently not true.
I made some quick changes to improve the situation.
What do you think @mortberg , @ecavallo ?

@felixwellen
Copy link
Collaborator Author

I guess it would make sense to say that cubical checks with whatever version of agda we use for testing.

@mortberg
Copy link
Collaborator

There was some discussion about this some time ago and the conclusion was that the library should compile on the latest official release of Agda. I think that's still the case or was there some new release that I'm not aware of?

@felixwellen
Copy link
Collaborator Author

Ok, that's easier. Now I just want to change 'development' to 'official release'.

@felixwellen felixwellen marked this pull request as ready for review February 10, 2022 15:58
@felixwellen felixwellen changed the title Don't say cubical compiles with the latest agda version Don't claim cubical compiles with the latest agda development version Feb 10, 2022
Copy link
Collaborator

@mortberg mortberg left a comment

Choose a reason for hiding this comment

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

Two tiny suggestions

README.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
felixwellen and others added 2 commits February 10, 2022 17:12
Co-authored-by: Anders Mörtberg <andersmortberg@gmail.com>
Co-authored-by: Anders Mörtberg <andersmortberg@gmail.com>
@mortberg mortberg self-requested a review February 10, 2022 16:16
@mortberg
Copy link
Collaborator

I guess we can merge now without waiting for the CI to finish?

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.

None yet

2 participants