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

Nick/neos 271 add configuration for tls certs for api #479

Merged

Conversation

nickzelei
Copy link
Member

No description provided.

Copy link

vercel bot commented Nov 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
neosync-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 2, 2023 5:37pm

Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Merging #479 (9839d06) into main (ac8f87d) will decrease coverage by 0.34%.
Report is 1 commits behind head on main.
The diff coverage is 65.38%.

❗ Current head 9839d06 differs from pull request most recent head ded99cc. Consider uploading reports for the commit ded99cc to get more accurate results

@@            Coverage Diff             @@
##             main     #479      +/-   ##
==========================================
- Coverage   45.71%   45.38%   -0.34%     
==========================================
  Files          42       42              
  Lines        3224     3241      +17     
==========================================
- Hits         1474     1471       -3     
- Misses       1589     1604      +15     
- Partials      161      166       +5     
Files Coverage Δ
...internal/temporal/client-manager/manager-client.go 81.25% <65.38%> (-6.04%) ⬇️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nickzelei nickzelei merged commit 1bfbeec into main Nov 2, 2023
8 checks passed
@nickzelei nickzelei deleted the nick/neos-271-add-configuration-for-tls-certs-for-api branch November 2, 2023 17:39
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