Skip to content

fix: Cannot convert date value to AST #278

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

Merged
merged 5 commits into from
Jul 10, 2020
Merged

Conversation

igdianov
Copy link
Collaborator

No description provided.

@igdianov igdianov self-assigned this Jul 10, 2020
@igdianov igdianov added the bug label Jul 10, 2020
@codecov
Copy link

codecov bot commented Jul 10, 2020

Codecov Report

Merging #278 into master will increase coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #278      +/-   ##
============================================
+ Coverage     74.02%   74.04%   +0.02%     
- Complexity     1070     1071       +1     
============================================
  Files            72       72              
  Lines          4754     4758       +4     
  Branches        680      681       +1     
============================================
+ Hits           3519     3523       +4     
+ Misses          934      929       -5     
- Partials        301      306       +5     
Impacted Files Coverage Δ Complexity Δ
...ventures/graphql/jpa/query/schema/JavaScalars.java 47.41% <66.66%> (+0.36%) 6.00 <0.00> (ø)
...hql/jpa/query/schema/impl/JpaPredicateBuilder.java 60.85% <0.00%> (+0.22%) 115.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e1421ca...aef5532. Read the comment docs.

@igdianov igdianov merged commit 06379c7 into master Jul 10, 2020
@igdianov igdianov deleted the fix-ast-date-serialize-error branch July 10, 2020 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant