Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Oct 23, 2016

This fixes bug #704.

The issue isn't so much using that symbol after /, but that scala was picking it up as part of the expansion of the macro quasiquote at Uri.scala:124.

Also fixes this bug in QValue.scala, just in case.

@codecov-io
Copy link

codecov-io commented Oct 23, 2016

Current coverage is 62.38% (diff: 100%)

Merging #741 into master will increase coverage by 0.18%

@@             master       #741   diff @@
==========================================
  Files           208        208          
  Lines          4416       4416          
  Methods        3968       3968          
  Messages          0          0          
  Branches        442        442          
==========================================
+ Hits           2747       2755     +8   
+ Misses         1669       1661     -8   
  Partials          0          0          

Powered by Codecov. Last update a46e49c...952036f

Copy link
Member

@rossabaker rossabaker left a comment

Choose a reason for hiding this comment

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

Ah, that makes sense! Thank you!

I'll plan on cherry-picking this onto release-0.14.x so we can get it out as a bugfix.

I'll cherry-pick this so we can get it out in the next 0.14 maintenance release.

@bryce-anderson
Copy link
Member

Sorry, this escaped me. Looks good! 👍

@rossabaker
Copy link
Member

Cherry-picked and publishing as 0.14.11. Thanks!

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.

3 participants