Fix Windows build

[chapulina]

Signed-off-by: Louise Poubel [email protected]

🦟 Bug fix

Fixes #1030

Summary

Renamed reserved word.

My bad for merging #1003 without passing Windows CI, but in my defense, #990 is making PR review very difficult.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

[codecov]

Codecov Report

Merging #1033 (e6a72d3) into main (1e1d0ba) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1033      +/-   ##
==========================================
+ Coverage   63.27%   63.30%   +0.03%     
==========================================
  Files         239      239              
  Lines       19464    19464              
==========================================
+ Hits        12316    12322       +6     
+ Misses       7148     7142       -6     

Impacted Files	Coverage Δ
...int_position_controller/JointPositionController.cc	75.19% <ø> (ø)
src/SimulationRunner.cc	93.92% <0.00%> (+1.01%)	⬆️

---

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 1e1d0ba...e6a72d3. Read the comment docs.

[arjo129]

Looks like its building now, although there seem to be other issues with the windows build. In particular there is some wierd "invalid vector subscript" error in the test case. My bad also for not checking the Windows build.

[chapulina]

Looks like its building now, although there seem to be other issues with the windows build

Yup, that's the current state of the Windows build 🙃 Thanks for the review!