Skip to content

Conversation

@wvandeun
Copy link
Contributor

In #409 we have split up the node.py file into multiple files.
This created an issue with the import path of the Attribute class.

@wvandeun wvandeun self-assigned this Jun 18, 2025
@wvandeun
Copy link
Contributor Author

Related to opsmill/infrahub-ansible#228

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 18, 2025

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: a3ad656
Status: ✅  Deploy successful!
Preview URL: https://f99ba840.infrahub-sdk-python.pages.dev
Branch Preview URL: https://wvd-fix-attribute-import-pat.infrahub-sdk-python.pages.dev

View logs

@wvandeun wvandeun force-pushed the wvd-fix-attribute-import-path branch from c5aacc3 to a3ad656 Compare June 18, 2025 19:02
@codecov
Copy link

codecov bot commented Jun 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@           Coverage Diff           @@
##           stable     #448   +/-   ##
=======================================
  Coverage   75.57%   75.57%           
=======================================
  Files         100      100           
  Lines        8750     8751    +1     
  Branches     1704     1704           
=======================================
+ Hits         6613     6614    +1     
  Misses       1661     1661           
  Partials      476      476           
Flag Coverage Δ
integration-tests 34.63% <0.00%> (-0.01%) ⬇️
python-3.10 47.84% <0.00%> (-0.03%) ⬇️
python-3.11 47.84% <0.00%> (-0.03%) ⬇️
python-3.12 47.84% <0.00%> (-0.01%) ⬇️
python-3.13 47.82% <0.00%> (-0.03%) ⬇️
python-3.9 46.51% <0.00%> (-0.03%) ⬇️
python-filler-3.12 25.28% <100.00%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
infrahub_sdk/node/__init__.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wvandeun wvandeun merged commit cefda84 into stable Jun 19, 2025
19 checks passed
@wvandeun wvandeun deleted the wvd-fix-attribute-import-path branch June 19, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants