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

Cherry pick from @HoshinoTented's branch #617

Merged
merged 5 commits into from
Oct 29, 2022
Merged

Cherry pick from @HoshinoTented's branch #617

merged 5 commits into from
Oct 29, 2022

Conversation

ice1000
Copy link
Member

@ice1000 ice1000 commented Oct 29, 2022

Hoshino Tented

We cannot solve metas in result type from clauses, because when we're in the clauses, the result type is substituted, and it doesn't make sense to solve a "substituted meta"
In the future, we may generate a "constant" meta and try to solve it if the result type is a pure meta.

@ice1000 ice1000 added this to the v0.23 milestone Oct 29, 2022
bors bot added a commit that referenced this pull request Oct 29, 2022
617: Cherry pick from @HoshinoTented's branch r=ice1000 a=ice1000

Hoshino Tented

bors r+

Co-authored-by: ice1000 <ice1000kotlin@foxmail.com>
@bors
Copy link
Contributor

bors bot commented Oct 29, 2022

Build failed:

  • gradle-check

@ice1000
Copy link
Member Author

ice1000 commented Oct 29, 2022

bors r+

bors bot added a commit that referenced this pull request Oct 29, 2022
617: Cherry pick from @HoshinoTented's branch r=ice1000 a=ice1000

Hoshino Tented

We cannot solve metas in result type from clauses, because when we're in the clauses, the result type is substituted, and it doesn't make sense to solve a "substituted meta"
In the future, we may generate a "constant" meta and try to solve it if the result type is a pure meta.

Co-authored-by: ice1000 <ice1000kotlin@foxmail.com>
@bors
Copy link
Contributor

bors bot commented Oct 29, 2022

Canceled.

@ice1000
Copy link
Member Author

ice1000 commented Oct 29, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 29, 2022

Build succeeded:

@codecov
Copy link

codecov bot commented Oct 29, 2022

Codecov Report

Merging #617 (0a60e78) into main (da190c1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main     #617   +/-   ##
=========================================
  Coverage     79.89%   79.90%           
- Complexity     2796     2797    +1     
=========================================
  Files           228      228           
  Lines          8850     8852    +2     
  Branches       1097     1098    +1     
=========================================
+ Hits           7071     7073    +2     
  Misses         1192     1192           
  Partials        587      587           
Impacted Files Coverage Δ
base/src/main/java/org/aya/core/term/Term.java 96.40% <100.00%> (ø)
base/src/main/java/org/aya/tyck/LittleTyper.java 72.72% <100.00%> (+0.35%) ⬆️
base/src/main/java/org/aya/tyck/StmtTycker.java 87.68% <100.00%> (+0.06%) ⬆️
base/src/main/java/org/aya/tyck/pat/PatTycker.java 90.36% <100.00%> (ø)

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

@bors bors bot merged commit 8726d0d into main Oct 29, 2022
@bors bors bot deleted the aboo branch October 29, 2022 04:10
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.

None yet

1 participant