Avoid Io.hh header name clash

[hidmic]

🦟 Bug fix

Fixes #470.

Summary

More of a hunch than a fix. I suspect neither Mac OS nor Windows do case sensitive header look up, which creates a clash between the Io.hh file and the generated io.hh file.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

[codecov]

Codecov Report

Merging #471 (376d299) into main (9ece5b5) will not change coverage.
The diff coverage is n/a.

❗ Current head 376d299 differs from pull request most recent head 769a992. Consider uploading reports for the commit 769a992 to get more accurate results

@@           Coverage Diff           @@
##             main     #471   +/-   ##
=======================================
  Coverage   81.07%   81.07%           
=======================================
  Files          79       79           
  Lines        9679     9679           
=======================================
  Hits         7847     7847           
  Misses       1832     1832           

Impacted Files	Coverage Δ
io/include/gz/common/DataFrame.hh	76.27% <ø> (ø)
io/include/gz/common/IOBase.hh	100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[mjcarroll]

Yep, this is very likely the fix.

[hidmic]

Would you mind releasing this @mjcarroll ?

[mjcarroll]

Ah yeah, in order for it to be released, we need to target something other than main. Can this land on garden?

[scpeters]

an alternative is to nest the headers of components in a subfolder like gz/common/io/Io.hh

[hidmic]

There was no fundamental reason to name that header Io.hh, so no biggie.