Skip to content

Merge stable into develop - #1262

Open
infrahub-github-bot-app[bot] wants to merge 1 commit into
developfrom
stable
Open

Merge stable into develop#1262
infrahub-github-bot-app[bot] wants to merge 1 commit into
developfrom
stable

Conversation

@infrahub-github-bot-app

@infrahub-github-bot-app infrahub-github-bot-app Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Merging stable into develop after merging pull request #1253.


Summary by cubic

Clarifies infrahubctl graphql query-report help text and docs so users understand what the report means and when results differ. No runtime behavior changes.

  • Help: name now clearly refers to the query defined under queries in .infrahub.yml. --branch notes that uniqueness depends on the schema, so results can differ by branch.
  • Docs: Explain how the tool determines whether a query targets unique nodes and why that matters for artifact/generator regeneration.

Written for commit c893a98. Summary will update on new commits.

Review in cubic

…mes from (#1253)

The command's help said only that it runs a query through
InfrahubGraphQLQueryReport, which names an internal type and tells a user
nothing about what the answer means.

Describe the consequence first: artifact and generator definitions use the
verdict to decide how much to regenerate when data changes. State the rule
that produces it, note that NAME comes from the queries section of
.infrahub.yml, and record on --branch that the answer is branch-dependent
because uniqueness constraints live in the schema.
@infrahub-github-bot-app
infrahub-github-bot-app Bot requested a review from a team as a code owner August 20, 2026 13:55
@github-actions github-actions Bot added the type/documentation Improvements or additions to documentation label Aug 20, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: c893a98
Status: ✅  Deploy successful!
Preview URL: https://354922fa.infrahub-sdk-python.pages.dev

View logs

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

❗ There is a different number of reports uploaded between BASE (d0537e5) and HEAD (c893a98). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (d0537e5) HEAD (c893a98)
integration-tests 1 0
@@             Coverage Diff             @@
##           develop    #1262      +/-   ##
===========================================
- Coverage    84.16%   78.06%   -6.10%     
===========================================
  Files          147      147              
  Lines        13045    13045              
  Branches      1930     1930              
===========================================
- Hits         10979    10184     -795     
- Misses        1503     2310     +807     
+ Partials       563      551      -12     
Flag Coverage Δ
integration-tests ?
python-3.10 56.99% <ø> (+0.01%) ⬆️
python-3.11 56.99% <ø> (ø)
python-3.12 56.97% <ø> (ø)
python-3.13 56.99% <ø> (+0.01%) ⬆️
python-3.14 56.97% <ø> (ø)
python-filler-3.12 23.68% <ø> (ø)

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

Files with missing lines Coverage Δ
infrahub_sdk/ctl/graphql.py 91.05% <ø> (ø)

... and 34 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant