einspect icon indicating copy to clipboard operation
einspect copied to clipboard

Update usages of id() to api.address()

Open ionite34 opened this issue 1 year ago • 1 comments

Improvements

  • Updated all usages of the id() function to api.address() to prevent issues when id() is replaced and to better support other python implementations where id() is not the memory address.

ionite34 avatar May 05 '23 07:05 ionite34

Codecov Report

Merging #98 (1e35daf) into main (640e7bf) will increase coverage by 0.01%. The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
+ Coverage   92.54%   92.55%   +0.01%     
==========================================
  Files          51       51              
  Lines        2801     2803       +2     
==========================================
+ Hits         2592     2594       +2     
  Misses        209      209              
Flag Coverage Δ
unittests 92.55% <100.00%> (+0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/einspect/__init__.py 100.00% <100.00%> (ø)
src/einspect/_patch.py 100.00% <100.00%> (ø)
src/einspect/structs/py_set.py 100.00% <100.00%> (ø)
src/einspect/structs/py_type.py 87.81% <100.00%> (+0.08%) :arrow_up:
src/einspect/views/view_base.py 92.93% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 640e7bf...1e35daf. Read the comment docs.

codecov[bot] avatar May 05 '23 07:05 codecov[bot]