Skip to content

Hub 0.10.2#114

Merged
pbrassel merged 6 commits into
mainfrom
hub-0.10.2
Jun 12, 2026
Merged

Hub 0.10.2#114
pbrassel merged 6 commits into
mainfrom
hub-0.10.2

Conversation

@pbrassel

@pbrassel pbrassel commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Added display_name field support for projects and analyses, enabling enhanced labeling and identification.
  • Chores

    • Bumped version to 0.2.16.
    • Updated test hub version to 0.10.2.

@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR adds display_name field support to projects and analyses throughout the core client, updates BaseClient type annotations to explicitly accept None values, and bumps version to 0.2.16 with corresponding test environment updates.

Changes

Display Name Feature and Type Safety

Layer / File(s) Summary
BaseClient type safety improvements
flame_hub/_base_client.py
Query-builder helpers (build_page_params, build_filter_params, build_sort_params, build_include_params, build_field_params) and resource-fetching methods (_get_all_resources, _find_all_resources, _get_single_resource) are updated with explicit | None type annotations. BaseClient.__init__ auth parameter is re-annotated to allow None, and explicit httpx.Timeout(5, read=20) configuration is removed from default client creation.
Display name data model fields
flame_hub/_core_client.py
CreateProject and CreateAnalysis gain optional display_name: str | None fields. UpdateProject and UpdateAnalysis gain display_name: str | None | UNSET_T fields for partial updates.
CoreClient display name method integration
flame_hub/_core_client.py
CoreClient.create_project(), update_project(), create_analysis(), and update_analysis() methods now accept display_name parameters and wire them into their respective request models.
Test coverage for display name updates
tests/test_core.py
test_update_project and test_update_analysis now pass display_name in update calls and assert the field is persisted and returned case-insensitively.
Release version updates
pyproject.toml, .env.test
Project version bumped from 0.2.15 to 0.2.16; test environment hub version updated from 0.10.1 to 0.10.2.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • PrivateAIM/hub-python-client#113: Prior PR updates the same test cases (test_update_project and test_update_analysis) to assert name case-insensitively; this PR extends those tests to additionally verify the new display_name field using the same pattern.

Poem

🐰 A display name for all to see,
Projects and analyses wearing labels free,
Type hints now guard against the null,
And tests confirm each field in full!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Hub 0.10.2' is vague and fails to describe the actual changes made in the pull request, which include adding display_name support for projects/analyses, updating type hints, and bumping the version. Use a more descriptive title that conveys the main change, such as 'Add display_name support for projects and analyses' or 'Add display_name field to project and analysis models'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hub-0.10.2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pbrassel pbrassel merged commit b425f76 into main Jun 12, 2026
6 of 7 checks passed
@pbrassel pbrassel deleted the hub-0.10.2 branch June 12, 2026 09:58
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.

1 participant