Skip to content

Conversation

fthomas
Copy link
Owner

@fthomas fthomas commented Jan 9, 2020

No description provided.

howyp and others added 11 commits February 3, 2019 22:29
 Move scala-xml related code to separate module
This makes the scalaxml module only available for the JVM because
scala-xml is not available for Scala Native and fails with linking
errors on Scala.js:
```
Referring to non-existent class javax.xml.parsers.SAXParserFactory$
   called from scala.xml.factory.XMLLoader.parser()javax.xml.parsers.SAXParser
   called from scala.xml.XML$.parser()javax.xml.parsers.SAXParser
   called from scala.xml.factory.XMLLoader.loadString(java.lang.String)scala.xml.Node
   called from scala.xml.XML$.loadString(java.lang.String)scala.xml.Node
   called from eu.timepit.refined.scalaxml.string$Xml$.$$anonfun$xmlValidate$1(java.lang.String)scala.xml.Elem
   called from eu.timepit.refined.scalaxml.string$Xml$.xmlValidate()eu.timepit.refined.api.Validate
```
@codecov
Copy link

codecov bot commented Jan 9, 2020

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 93.28%. Comparing base (c25374f) to head (63a5583).
⚠️ Report is 745 commits behind head on master.

Files with missing lines Patch % Lines
.../main/scala/eu/timepit/refined/scalaxml/util.scala 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #713      +/-   ##
==========================================
+ Coverage   92.39%   93.28%   +0.89%     
==========================================
  Files          63       65       +2     
  Lines         815      804      -11     
  Branches        6       10       +4     
==========================================
- Hits          753      750       -3     
+ Misses         62       54       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fthomas
Copy link
Owner Author

fthomas commented Sep 26, 2020

Before merging, this should happen on the series/0.10 branch before:

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.

2 participants