Skip to content

bumping whenever dependency#1085

Open
mhdask wants to merge 10 commits into
opsmill:infrahub-developfrom
mhdask:whenever-bump
Open

bumping whenever dependency#1085
mhdask wants to merge 10 commits into
opsmill:infrahub-developfrom
mhdask:whenever-bump

Conversation

@mhdask

@mhdask mhdask commented Jun 17, 2026

Copy link
Copy Markdown

Why

Started working on packaging infrahub for nix/NixOS - For that, some dependency bumping is needed, to align with the other versions of dependencies in nixpkgs
This bumps whenever to 0.10.0

What changed

  • Bumped whenever to 0.10.0
  • Due to API change, had to adjust usage of rounding for ZonedDateTime
  • 0.10.0deprecates py_datetime in favour of to_stdlib

How to review

  • Needs to be tested against infrahub-testcontainers from release-1.10 due to dependencies against prefect-client

Checklist

  • Tests added/updated
  • Changelog entry added (uv run towncrier create ...)
  • External docs updated (if user-facing or ops-facing change)
  • Internal .md docs updated (internal knowledge and AI code tools knowledge)

Summary by cubic

Bump whenever to 0.10.x and update timestamp calls to the new API. Also fix upsert with CoreNumberPool-backed HFIDs and make repo detection work in Git worktrees.

  • Dependencies

    • Update whenever to >=0.10.0,<0.11.0.
  • Bug Fixes

    • Upsert: save/create(allow_upsert=True) now raises a clear ValidationError if the HFID includes a CoreNumberPool-sourced attribute, instead of failing with a backend error.
    • Repository: Detect existing repos when .git is a worktree gitlink file to avoid re-init crashes.

Written for commit 483f855. Summary will update on new commits.

Review in cubic

petercrocker and others added 8 commits May 16, 2026 15:45
GitRepoManager.initialize_repo() checked for `.git` with `.is_dir()`, which
misses git worktrees (`.git` is a file containing `gitdir: ...`). The
existing-repo branch was skipped and `Repo.init` then crashed with
`FileExistsError` trying to mkdir the gitlink file. This made `infrahubctl
transform` (and any other CLI relying on branch auto-detection) unusable from
a worktree.

Switch the check to `.exists()` — dulwich's `Repo(path)` already resolves
the gitlink to the real controldir, and `porcelain.active_branch` returns
the worktree's branch correctly.
Bumps [starlette](https://github.com/Kludex/starlette) from 0.49.3 to 1.0.1.
- [Release notes](https://github.com/Kludex/starlette/releases)
- [Changelog](https://github.com/Kludex/starlette/blob/main/docs/release-notes.md)
- [Commits](Kludex/starlette@0.49.3...1.0.1)

---
updated-dependencies:
- dependency-name: starlette
  dependency-version: 1.0.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [pyarrow](https://github.com/apache/arrow) from 22.0.0 to 23.0.1.
- [Release notes](https://github.com/apache/arrow/releases)
- [Commits](apache/arrow@apache-arrow-22.0.0...apache-arrow-23.0.1)

---
updated-dependencies:
- dependency-name: pyarrow
  dependency-version: 23.0.1
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
….0.1

chore(deps): bump pyarrow from 22.0.0 to 23.0.1
…1.0.1

chore(deps): bump starlette from 0.49.3 to 1.0.1
fix(repository): detect existing repo when .git is a worktree gitlink
@mhdask mhdask requested a review from a team as a code owner June 17, 2026 19:00

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

@mhdask

mhdask commented Jun 24, 2026

Copy link
Copy Markdown
Author

Hmm seems that infrahub-develop branch has fallen behind table

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.

4 participants