Skip to content

Merge develop into infrahub-develop - #1267

Merged
infrahub-github-bot-app[bot] merged 6 commits into
infrahub-developfrom
develop
Aug 24, 2026
Merged

Merge develop into infrahub-develop#1267
infrahub-github-bot-app[bot] merged 6 commits into
infrahub-developfrom
develop

Conversation

@infrahub-github-bot-app

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

Copy link
Copy Markdown
Contributor

Merging develop into infrahub-develop after merging pull request #1262.


Summary by cubic

Clarifies what infrahubctl graphql query-report reports and why it matters, tightens type-checker suppressions, and refreshes HTTP/2 dependencies. No runtime behavior changes; improved help/docs and cleaned suppressions may surface new type errors in CI.

  • CLI/docs: Explain that query-report determines if a query targets unique nodes, how artifacts/generators use the verdict, when a query is unique, that NAME comes from .infrahub.yml, and that results can differ by --branch.
  • Type checking: Narrow over-broad suppressions in pyproject.toml (use mypy’s prop-decorator, drop unused arg-type/fixture/no-matching-overload ignores, update violation counts).
  • Dependencies: Bump h2 to 4.4.1 and hpack to 4.2.0 in uv.lock (indirect); no code changes expected.

Written for commit f65df4a. 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.
The generated read models were suppressing all of mypy's `misc` code to
silence one decorated-property error. mypy split that diagnostic into its
own `prop-decorator` code in 1.11, the version pinned here, so scope the
override to it.

Verified by removing each suppression and re-running the checker, which
also surfaced three dead entries (mypy `arg-type` on utils, ty
`invalid-argument-type` on test fixtures, ty `no-matching-overload`) and
two stale violation counts.
…ressions

chore: narrow over-broad mypy and ty error suppressions
Merge stable into develop
@infrahub-github-bot-app
infrahub-github-bot-app Bot requested a review from a team as a code owner August 24, 2026 06:02
@github-actions github-actions Bot added the type/documentation Improvements or additions to documentation label Aug 24, 2026
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@                Coverage Diff                @@
##           infrahub-develop    #1267   +/-   ##
=================================================
  Coverage             84.16%   84.16%           
=================================================
  Files                   147      147           
  Lines                 13045    13045           
  Branches               1930     1930           
=================================================
  Hits                  10979    10979           
  Misses                 1503     1503           
  Partials                563      563           
Flag Coverage Δ
integration-tests 39.01% <ø> (ø)
python-3.10 56.97% <ø> (ø)
python-3.11 56.97% <ø> (-0.02%) ⬇️
python-3.12 56.99% <ø> (ø)
python-3.13 56.97% <ø> (-0.02%) ⬇️
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% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

dependabot Bot and others added 2 commits August 24, 2026 09:36
Bumps [h2](https://github.com/python-hyper/h2) from 4.3.0 to 4.4.1.
- [Changelog](https://github.com/python-hyper/h2/blob/master/CHANGELOG.rst)
- [Commits](python-hyper/h2@v4.3.0...v4.4.1)

---
updated-dependencies:
- dependency-name: h2
  dependency-version: 4.4.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Merge stable into develop
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: f65df4a
Status: ✅  Deploy successful!
Preview URL: https://9c408ba9.infrahub-sdk-python.pages.dev
Branch Preview URL: https://develop.infrahub-sdk-python.pages.dev

View logs

@infrahub-github-bot-app
infrahub-github-bot-app Bot merged commit 73899cb into infrahub-develop Aug 24, 2026
33 checks passed
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