coverallsapp / coverage-reporter

Self-contained, universal coverage uploader binary. Under development.
https://coveralls.io
MIT License
49 stars 17 forks source link

fix: use Crystal 1.7.3 #66

Closed mrexox closed 1 year ago

mrexox commented 1 year ago

Closes https://github.com/coverallsapp/coverage-reporter/issues/65

:zap: Summary

In Crystal 1.8 they use pcre2 (PR), and it raises an exception when parsing non UTF-8 text. It's hard to find out what exactly was wrong (as said here, PCRE2 devs don't remember that :thinking:), so let's leave it for the later work. This PR downgrades Crystal version where everything worked just fine.

:ballot_box_with_check: Checklist

coveralls-official[bot] commented 1 year ago

Pull Request Test Coverage Report for Build 4817258498


Totals Coverage Status
Change from base Build 4809347986: 0.0%
Covered Lines: 720
Relevant Lines: 795

💛 - Coveralls