fix: don't transform_href in __getstate__

[gadomski]

Related Issue(s):

• Closes New pickling methods loads all absolute links #1336
• Relates to Object serialization should not make network requests by default #960

Description:

__getstate__ was doing too much work by trying to resolve the root. @jsignell we'll want to release a PATCH once this merges, as this will be a good fix to get out in the wild.

PR Checklist:

• pre-commit hooks pass locally
• Tests pass (run scripts/test)
• Documentation has been updated to reflect changes, if applicable
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.22%. Comparing base (3432392) to head (45577b0).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1337   +/-   ##
=======================================
  Coverage   91.22%   91.22%           
=======================================
  Files          51       51           
  Lines        7043     7043           
  Branches     1008     1008           
=======================================
  Hits         6425     6425           
  Misses        443      443           
  Partials      175      175           

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

[jsignell]

LGTM thanks Pete 🚀