Conversation

phamconganh

No description provided.

nextExp.text == "on" || nextExp.text == "1" ? 1 : 0));
} catch (_) {}

Choose a reason for hiding this comment

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

Can you describe a little bit about why we're doing this and ignoring the caught error? Is there a issue you can reference to describe what the justification for this PR is?

Copy link
Author

Choose a reason for hiding this comment

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

it was a long time ago so i don't remember clearly. there was a case that caused the init font reading to fail, i added it so it wouldn't affect the rendering

@codecovCodecov

Codecov Report

Attention: coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 64.08%. Comparing base (2814407) to head (8aca949).

Files with missing lines%Lines
lib/src/css_parser.dart0.00%3 Missing⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1379   +/-   ##
=======================================
  Coverage   64.08%   64.08%           
=======================================
  Files          39       39           
  Lines        3032     3032           
=======================================
  Hits         1943     1943           
  Misses       1089     1089           

☔ 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.

Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
Status: Todo

Successfully merging this pull request may close these issues.