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

Higher inductive types #669

Merged
merged 26 commits into from
Nov 16, 2022
Merged

Higher inductive types #669

merged 26 commits into from
Nov 16, 2022

Conversation

ice1000
Copy link
Member

@ice1000 ice1000 commented Nov 16, 2022

Whoo-hoo!

Completely drop conditions now.

@ice1000 ice1000 added this to the v0.24 milestone Nov 16, 2022
@ice1000 ice1000 added the hits Higher inductive types label Nov 16, 2022
@codecov
Copy link

codecov bot commented Nov 16, 2022

Codecov Report

Merging #669 (58020ab) into main (7439003) will decrease coverage by 0.21%.
The diff coverage is 84.25%.

@@             Coverage Diff              @@
##               main     #669      +/-   ##
============================================
- Coverage     78.74%   78.52%   -0.22%     
+ Complexity     2875     2858      -17     
============================================
  Files           260      260              
  Lines          9423     9426       +3     
  Branches       1167     1166       -1     
============================================
- Hits           7420     7402      -18     
- Misses         1382     1396      +14     
- Partials        621      628       +7     
Impacted Files Coverage Δ
.../src/main/java/org/aya/concrete/stmt/TeleDecl.java 90.54% <ø> (ø)
...rc/main/java/org/aya/concrete/visitor/StmtOps.java 0.00% <0.00%> (ø)
base/src/main/java/org/aya/core/def/CtorDef.java 80.00% <ø> (ø)
...ase/src/main/java/org/aya/core/pat/PatMatcher.java 74.13% <0.00%> (-6.90%) ⬇️
base/src/main/java/org/aya/core/serde/SerPat.java 75.00% <ø> (-6.25%) ⬇️
...ase/src/main/java/org/aya/core/term/MatchTerm.java 12.50% <ø> (ø)
...rc/main/java/org/aya/core/visitor/DefConsumer.java 36.58% <0.00%> (-1.88%) ⬇️
...e/src/main/java/org/aya/core/visitor/Expander.java 31.57% <0.00%> (ø)
...src/main/java/org/aya/core/visitor/TermFolder.java 63.63% <0.00%> (+2.27%) ⬆️
base/src/main/java/org/aya/tyck/ExprTycker.java 86.77% <ø> (ø)
... and 31 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ice1000 ice1000 requested a review from imkiva November 16, 2022 19:51
@ice1000
Copy link
Member Author

ice1000 commented Nov 16, 2022

I guess merge for now (sorry about Hoshino's commits)

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 16, 2022

Build succeeded:

@bors bors bot merged commit 7686dc1 into main Nov 16, 2022
@bors bors bot deleted the suede branch November 16, 2022 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hits Higher inductive types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants