Skip to content

Conversation

mcarey1590
Copy link

Fixes EditServer by passing in the payload

Copy link

mergify bot commented Jan 10, 2025

Welcome to our open-source project! 💘

@mcarey1590
Copy link
Author

👋🏼 I'm working on a comprehensive terraform provider for postmark. Thanks for the work you have done on this library!

@mcarey1590 mcarey1590 changed the title fix: set server payload on PUT request feat: update server requests Jan 12, 2025
@mrz1836 mrz1836 added feature Any new significant addition update General updates labels Jan 13, 2025
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 92.10526% with 3 lines in your changes missing coverage. Please review.

Project coverage is 92.58%. Comparing base (1806d09) to head (f36b361).
Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
servers.go 92.10% 2 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   91.31%   92.58%   +1.26%     
==========================================
  Files          13       13              
  Lines         645      674      +29     
==========================================
+ Hits          589      624      +35     
+ Misses         48       41       -7     
- Partials        8        9       +1     
Flag Coverage Δ
unittests 92.58% <92.10%> (+1.26%) ⬆️

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

Files with missing lines Coverage Δ
servers.go 68.00% <92.10%> (+33.21%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1806d09...f36b361. Read the comment docs.

@mrz1836
Copy link
Owner

mrz1836 commented Jan 13, 2025

Thank you for your contribution @mcarey1590

@mrz1836 mrz1836 merged commit 64ba815 into mrz1836:master Jan 13, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Any new significant addition update General updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants