Skip to content

feat: adds client dependency injection to model#8

Merged
becelot merged 2 commits intomainfrom
feat/dependency_injection
Nov 9, 2021
Merged

feat: adds client dependency injection to model#8
becelot merged 2 commits intomainfrom
feat/dependency_injection

Conversation

@becelot
Copy link
Copy Markdown
Member

@becelot becelot commented Nov 9, 2021

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 9, 2021

Codecov Report

Merging #8 (4cc19c9) into main (172d86a) will increase coverage by 0.51%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   86.86%   87.37%   +0.51%     
==========================================
  Files          23       23              
  Lines         685      689       +4     
==========================================
+ Hits          595      602       +7     
+ Misses         90       87       -3     
Flag Coverage Δ
unit 87.37% <90.90%> (+0.51%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
hostingde/hostingde.py 75.51% <0.00%> (ø)
hostingde/model/account.py 72.72% <66.66%> (-7.28%) ⬇️
hostingde/model/__init__.py 100.00% <100.00%> (+10.52%) ⬆️
hostingde/model/job.py 100.00% <100.00%> (ø)
hostingde/model/record.py 91.80% <100.00%> (ø)
hostingde/model/soa_values.py 100.00% <100.00%> (ø)
hostingde/model/sort.py 100.00% <100.00%> (ø)
hostingde/model/zone.py 92.85% <100.00%> (+1.94%) ⬆️
hostingde/model/zone_config.py 96.15% <100.00%> (ø)

@becelot becelot merged commit 07a9d52 into main Nov 9, 2021
@Seenox Seenox deleted the feat/dependency_injection branch June 30, 2022 11:23
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