Improve value renaming

[justinchuby]

Do not modify the value name if the new name is invalid. This makes the setter of Value.name safer in that things will not be left in an inconsistent state if a check fails

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.81%. Comparing base (49a361c) to head (9914dcc).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
+ Coverage   76.80%   76.81%   +0.01%     
==========================================
  Files          40       40              
  Lines        4915     4922       +7     
  Branches      978      980       +2     
==========================================
+ Hits         3775     3781       +6     
  Misses        856      856              
- Partials      284      285       +1     

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