Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add has_id_or_name to DomainIdentityMixin #373

Merged
merged 2 commits into from
Mar 27, 2024

Conversation

jooola
Copy link
Member

@jooola jooola commented Mar 27, 2024

  • Add a method to check if 2 domains are equal (only check for same ID or same name)
  • Adds the DomainIdentityMixin to models that were missing it.

@jooola jooola changed the title feat: add has_id_or_name to DomainIdentityMixin feat: add has_id_or_name to DomainIdentityMixin Mar 27, 2024
@jooola jooola requested a review from apricote March 27, 2024 14:37
@jooola jooola force-pushed the DomainIdentityMixin-has_id_or_name branch from fff17d4 to 07c6c24 Compare March 27, 2024 14:38
Copy link

codecov bot commented Mar 27, 2024

Codecov Report

Attention: Patch coverage is 95.65217% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 92.84%. Comparing base (6651b60) to head (fd5d804).

Files Patch % Lines
hcloud/core/domain.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #373      +/-   ##
==========================================
- Coverage   92.86%   92.84%   -0.02%     
==========================================
  Files          65       65              
  Lines        2872     2881       +9     
==========================================
+ Hits         2667     2675       +8     
- Misses        205      206       +1     

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

@jooola jooola force-pushed the DomainIdentityMixin-has_id_or_name branch from 07c6c24 to 2b92e44 Compare March 27, 2024 14:40
@jooola jooola merged commit 8facaf6 into hetznercloud:main Mar 27, 2024
10 checks passed
jooola pushed a commit that referenced this pull request Mar 27, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.34.0](v1.33.3...v1.34.0)
(2024-03-27)


### Features

* add `has_id_or_name` to `DomainIdentityMixin`
([#373](#373))
([8facaf6](8facaf6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
@jooola jooola deleted the DomainIdentityMixin-has_id_or_name branch March 27, 2024 15:27
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.

2 participants