refactor grids version testing

[gauravharsha]

No description provided.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0012150) to head (fbd71d8).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main        #6      +/-   ##
===========================================
+ Coverage   99.16%   100.00%   +0.83%     
===========================================
  Files           5         4       -1     
  Lines         240       222      -18     
===========================================
- Hits          238       222      -16     
+ Misses          2         0       -2     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Refactors the grids version consistency check used by the C++ application by removing the local version-string comparison helper and delegating version validation to a green::grids helper.

Changes:

• Removed compare_version_strings from green/ac/common_defs.h and deleted its associated unit test.
• Updated main.cpp to call green::grids::check_grids_version_in_hdf5(...) instead of doing an inline attribute read + comparison.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
test/ac_test.cpp	Removes the version string comparison test case that depended on the deleted helper.
src/green/ac/common_defs.h	Removes the inline version string comparison helper from a public header.
main.cpp	Replaces local HDF5 attribute/version check logic with a green::grids helper call.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[egull]

Totally fine like it is, or with copilot's changes.