Use "omit" instead of "null" to skip compose

[spetrosi]

Using null doesn't work because

Using null as suggested by current readme fails with the following error. An example run is here: https://github.com/redhat-cop/infra.leapp/actions/runs/19141825777

Run sclorg/testing-farm-as-github-action@v4
Error: [
  {
    "code": "invalid_type",
    "expected": "object",
    "received": "null",
    "path": [
      "environments",
      0,
      "os"
    ],
    "message": "Expected object, received null"
  }
]

I've tested my edits in https://github.com/spetrosi/infra.leapp/actions/runs/19170275013/job/54800766505

These changes are generated by Cursor AI

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.20%. Comparing base (1d700fe) to head (9b055c8).
⚠️ Report is 26 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
+ Coverage   91.58%   92.20%   +0.62%     
==========================================
  Files          14       14              
  Lines         867      911      +44     
  Branches      162      231      +69     
==========================================
+ Hits          794      840      +46     
+ Misses         73       64       -9     
- Partials        0        7       +7     

☔ View full report in Codecov by Harness.
📢 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[jamacku]

Thank you for the PR, but this is not right. You haven't updated the dist/ files, so you are still testing the original code.

But I know where the problem is.

[spetrosi]

It's shit code, sorry and thank you for looking at it.
Btw my successful run https://github.com/spetrosi/infra.leapp/actions/runs/19170275013/workflow#L199 used compose: null, not compose: omit as I thought, and It worked. So somewhat, my changes fix the issue.
My initial thought that null is a special value in yaml and it was not read as a string nor empty.

[phracek]

It's shit code, sorry and thank you for looking at it.
Btw my successful run https://github.com/spetrosi/infra.leapp/actions/runs/19170275013/workflow#L199 used compose: null, not compose: omit as I thought, and It worked. So somewhat, my changes fix the issue.
My initial thought that null is a special value in yaml and it was not read as a string nor empty.

@spetrosi Is this still valid? If not please close it.

[jamacku]

Closing, since this is no longer needed.

[spetrosi]

#326 (comment) it was pointed above that I didn't change dist files that are actually used when action is run, so my use of GH action from this branch was ignored. So this is not a real issue.
redhat-cop/infra.leapp#402