Move the function read_file from the ctl module to the SDK.

[dgarros]

Small fix to avoid having infrahub_sdk/yaml.py import a function from the ctl directory.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##           stable     #275      +/-   ##
==========================================
- Coverage   70.36%   70.35%   -0.02%     
==========================================
  Files          82       81       -1     
  Lines        7502     7499       -3     
  Branches     1391     1391              
==========================================
- Hits         5279     5276       -3     
  Misses       1849     1849              
  Partials      374      374              

Flag	Coverage Δ
integration-tests	22.45% <0.00%> (+<0.01%)	⬆️
python-3.10	45.52% <80.00%> (-0.07%)	⬇️
python-3.11	45.52% <80.00%> (-0.07%)	⬇️
python-3.12	45.52% <80.00%> (-0.07%)	⬇️
python-3.13	45.52% <80.00%> (-0.07%)	⬇️
python-3.9	45.52% <80.00%> (-0.07%)	⬇️
python-filler-3.12	24.00% <20.00%> (+0.04%)	⬆️

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

Files with missing lines	Coverage Δ
infrahub_sdk/ctl/exceptions.py	60.00% <ø> (-17.78%)	⬇️
infrahub_sdk/ctl/repository.py	81.57% <100.00%> (ø)
infrahub_sdk/ctl/utils.py	66.42% <100.00%> (ø)
infrahub_sdk/exceptions.py	80.23% <100.00%> (+0.96%)	⬆️
infrahub_sdk/utils.py	78.84% <100.00%> (+0.84%)	⬆️
infrahub_sdk/yaml.py	78.48% <100.00%> (-0.27%)	⬇️

[ogenstad]

LGTM, though I think we have a FileNotValidError exception class that we can delete from infrahub_sdk/ctl/exceptions.py as well as a cleanup.