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

Clear Garbage Code of PatTycker #732

Merged
merged 2 commits into from
Nov 25, 2022
Merged

Clear Garbage Code of PatTycker #732

merged 2 commits into from
Nov 25, 2022

Conversation

HoshinoTented
Copy link
Contributor

No description provided.

@HoshinoTented HoshinoTented added this to the v0.25 milestone Nov 25, 2022
@HoshinoTented HoshinoTented self-assigned this Nov 25, 2022
@codecov
Copy link

codecov bot commented Nov 25, 2022

Codecov Report

Merging #732 (016352a) into main (d2cd2f0) will decrease coverage by 0.05%.
The diff coverage is 98.30%.

@@             Coverage Diff              @@
##               main     #732      +/-   ##
============================================
- Coverage     81.48%   81.42%   -0.06%     
- Complexity     2999     3000       +1     
============================================
  Files           260      260              
  Lines          9574     9565       -9     
  Branches       1188     1188              
============================================
- Hits           7801     7788      -13     
- Misses         1118     1123       +5     
+ Partials        655      654       -1     
Impacted Files Coverage Δ
...va/org/aya/cli/library/json/LibraryConfigData.java 100.00% <ø> (ø)
base/src/main/java/org/aya/tyck/StmtTycker.java 87.68% <85.71%> (ø)
base/src/main/java/org/aya/tyck/ExprTycker.java 86.94% <100.00%> (-0.03%) ⬇️
base/src/main/java/org/aya/tyck/Tycker.java 100.00% <100.00%> (ø)
base/src/main/java/org/aya/tyck/pat/PatTycker.java 91.25% <100.00%> (-0.02%) ⬇️
...ase/src/main/java/org/aya/tyck/pat/TypedSubst.java 86.95% <0.00%> (-13.05%) ⬇️
base/src/main/java/org/aya/core/visitor/Subst.java 76.74% <0.00%> (-4.66%) ⬇️

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

@ice1000 ice1000 marked this pull request as ready for review November 25, 2022 20:32
@ice1000
Copy link
Member

ice1000 commented Nov 25, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 25, 2022

Build succeeded:

@bors bors bot merged commit ad0a182 into main Nov 25, 2022
@bors bors bot deleted the fix-pat branch November 25, 2022 20:35
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

2 participants