Skip to content

Conversation

@jooola
Copy link
Member

@jooola jooola commented Aug 12, 2025

This rename will reduce confusion in the future when reading the code. The ClientEntity name does not refer to anything.

This is scoped as a fix because the class is public.

@jooola jooola changed the title fix: renamed ClientEntityBase to BaseResourceClient fix: rename ClientEntityBase to BaseResourceClient Aug 12, 2025
@codecov
Copy link

codecov bot commented Aug 12, 2025

Codecov Report

❌ Patch coverage is 95.12195% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.01%. Comparing base (9096813) to head (0546dea).
⚠️ Report is 47 commits behind head on main.

Files with missing lines Patch % Lines
hcloud/core/client.py 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #532      +/-   ##
==========================================
- Coverage   96.07%   96.01%   -0.07%     
==========================================
  Files          64       64              
  Lines        2933     2938       +5     
==========================================
+ Hits         2818     2821       +3     
- Misses        115      117       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@jooola jooola marked this pull request as draft August 12, 2025 10:10
@jooola jooola force-pushed the refactor-rename-resource-base-client branch 2 times, most recently from 9da2c2f to 8f0572e Compare August 12, 2025 13:19
@jooola jooola changed the title fix: rename ClientEntityBase to BaseResourceClient fix: rename ClientEntityBase to ResourceClientBase Aug 14, 2025
@jooola jooola marked this pull request as ready for review August 14, 2025 12:40
@jooola jooola self-assigned this Aug 14, 2025
@jooola jooola requested a review from apricote August 14, 2025 12:49
@jooola jooola force-pushed the refactor-rename-resource-base-client branch from 148f6fb to 0546dea Compare August 14, 2025 13:49
@jooola
Copy link
Member Author

jooola commented Aug 14, 2025

I'll abuse my powers and merge this:

  • this class should not be used by anyone,
  • the changes are backward compatible,
  • this is mostly a deprecation warning and a search and replace.

@jooola jooola merged commit c3cd1c9 into main Aug 14, 2025
11 checks passed
@jooola jooola deleted the refactor-rename-resource-base-client branch August 14, 2025 13:57
@apricote
Copy link
Member

Looks good :)

@jooola jooola changed the title fix: rename ClientEntityBase to ResourceClientBase fix: rename ClientEntityBase to ResourceClientBase Sep 8, 2025
jooola added a commit that referenced this pull request Sep 8, 2025
<!-- section-start changelog -->
### Features

- add category property to server type (#549)

### Bug Fixes

- rename `ClientEntityBase` to `ResourceClientBase`  (#532)

<!-- section-end changelog -->

---

<details>
<summary><h4>PR by <a
href="https://github.com/apricote/releaser-pleaser">releaser-pleaser</a>
🤖</h4></summary>

If you want to modify the proposed release, add you overrides here. You
can learn more about the options in the docs.

## Release Notes

### Prefix / Start

This will be added to the start of the release notes.

```rp-prefix
```

### Suffix / End

This will be added to the end of the release notes.

```rp-suffix
```

</details>

---------

Co-authored-by: Hetzner Cloud Bot <>
Co-authored-by: jo <ljonas@riseup.net>
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