working on assigning s

[kurtbrose]

> c:\users\kurt\workspace\glom\glom\mutable.py(156)glomit()
-> if op == '[':
(Pdb) dest
ChainMap({T: {}, 'spec': Path()}, {T: {}, 'spec': (<glom.mutable.Assign object at 0x0000000002E73550>, S['foo'])}, {<class 'glom.core.Inspect'>: None, <class 'glom.core.Path'>: []}, {<function glom at 0x00000000031D1668>: <function _glom at 0x00000000031D1AC8>, <class 'glom.core.TargetRegistry'>: <glom.core.TargetRegistry object at 0x00000000031D2438>})```

not sure why scope parents seems to have that extra Path() thingie at the end

[codecov-io]

Codecov Report

Merging #71 into master will increase coverage by 0.16%.
The diff coverage is 96.2%.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   94.69%   94.85%   +0.16%     
==========================================
  Files          14       14              
  Lines        1545     1595      +50     
  Branches      234      241       +7     
==========================================
+ Hits         1463     1513      +50     
  Misses         47       47              
  Partials       35       35

Impacted Files	Coverage Δ
glom/test/test_mutable.py	98.24% <100%> (+0.03%)	⬆️
glom/mutable.py	98.88% <100%> (+0.06%)	⬆️
glom/test/test_path_and_t.py	99.07% <100%> (+0.17%)	⬆️
glom/core.py	96.86% <94.11%> (+0.12%)	⬆️

---

Continue to review full report at Codecov.

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

[mahmoud]

Added a couple more tests for more coverage, and with that I think we're ready! Merging!

[kurtbrose]

sweet! I'm happier with how it came out than the first few stabs -- passing parent scope as the target of the glom-eval that resolves the destination feels more in the spirit of glom than messing around with scope in the body of Assign.glom