@BenjaminNavarro FYI. This is still a work-in-progress, but I'm satisfied with the mechanics and it already passes all the existing tests, so if you want you can take a look.

I'm still going to try to minimize code duplication across ConstNodeRef and NodeRef, and I have to add more documentation in the quickstart and changelog.

Nice, didn't thought this was going to be taken care of so quickly.

To minimize code duplication I guess you could have either:

1. NodeRef inherits from ConstNodeRef and adds the state modifying functions
2. Have two separate classes, e.g NodeReadOperations and NodeWriteOperations and have ConstNodeRef inherit from the first and NodeRef from both.

I think 2. is what is used in Eigen, but that might require some CRTP to pull off. 1. is easy but some people don't like to use inheritance for these things;

Codecov Report

Merging #267 (06c955e) into master (bddb04b) will increase coverage by 0.04%.
The diff coverage is 98.73%.

❗ Current head 06c955e differs from pull request most recent head 9097d26. Consider uploading reports for the commit 9097d26 to get more accurate results

@@            Coverage Diff             @@
##           master     #267      +/-   ##
==========================================
+ Coverage   97.11%   97.16%   +0.04%     
==========================================
  Files          77       78       +1     
  Lines       17147    17757     +610     
==========================================
+ Hits        16653    17253     +600     
- Misses        494      504      +10     

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more