Skip to content

Conversation

@imrn99
Copy link
Collaborator

@imrn99 imrn99 commented Oct 17, 2024

  • create the new module
  • move DartIdentifier, NULL_DART_ID to it
  • move CMapBuilder a all related items to it

@codecov-commenter
Copy link

codecov-commenter commented Oct 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.67%. Comparing base (ab03601) to head (0ea6ca2).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   81.67%   81.67%           
=======================================
  Files          38       38           
  Lines        5342     5342           
=======================================
  Hits         4363     4363           
  Misses        979      979           

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

@imrn99 imrn99 marked this pull request as ready for review October 17, 2024 09:22
@imrn99 imrn99 mentioned this pull request Oct 17, 2024
5 tasks
@imrn99
Copy link
Collaborator Author

imrn99 commented Oct 18, 2024

postponing this due to change in approach. I'll probably do something similar when 3D is implemetned?

@imrn99 imrn99 closed this Oct 18, 2024
@imrn99 imrn99 deleted the extract-common-module branch October 18, 2024 09:35
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