Skip to content

Conversation

@tpoliaw
Copy link
Contributor

@tpoliaw tpoliaw commented Dec 8, 2025

  • Remove derives from types that no longer need them
  • Remove Links struct and associated fields
  • Reduce visibility of fields where possible

We're no longer exposing tiled data exactly as recieved so it
no longer needs to implement async-graphql's traits.
Make it easier to reason about which fields are being used and which are
likely to be removable.
@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 51.00%. Comparing base (9e4e542) to head (80ecc53).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/model/node.rs 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
- Coverage   52.64%   51.00%   -1.65%     
==========================================
  Files          13       13              
  Lines         585      600      +15     
==========================================
- Hits          308      306       -2     
- Misses        277      294      +17     

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

2 participants