DataOffload: fix generation of offload pragmas

[awnawab]

A small fix to how the global variable offload pragmas are generated which allows them to be split into multiple lines by fgen.

[github-actions]

Documentation for this branch can be viewed at https://sites.ecmwf.int/docs/loki/442/index.html

[mlange05]

Yes, makes sense and looks good to me. Approving now, but needs rebase once Pydantic version compatibility is fixed.

[reuterbal]

Thanks! The fix looks good and also personally I'm of the opinion that white space behind commas is not a luxury that should lay beyond us :)

The question is whether we wouldn't want to make commas itself a delimiter for chunking in JoinableStringList:

loki/loki/tools/strings.py

Line 57 in 72d35fb

_pattern_chunk_separator = re.compile(r'(\s|\)(?!%)|\n)')

This pattern governs the split points in strings when long lines need to be wrapped.

I have pushed a rebase to include the pydantic compatibility change.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.32%. Comparing base (72d35fb) to head (1433bea).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
- Coverage   93.32%   93.32%   -0.01%     
==========================================
  Files         212      212              
  Lines       40769    40769              
==========================================
- Hits        38047    38046       -1     
- Misses       2722     2723       +1     

Flag	Coverage Δ
lint_rules	96.39% <ø> (ø)
loki	93.27% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests