search

nosdav / server

NosDAV Server
https://nosdav.com/server
MIT License
9 stars 1 forks source link

[Ellipsis] Update README with Optional HTTPS Details #15

Closed ellipsis-dev[bot] closed 2 months ago

ellipsis-dev[bot] commented 2 months ago

This change addresses review comments left by @melvincarvalho on PR #13: Update README.md

:warning: We couldn't build/test your project to verify our changes. Add a Dockerfile to significantly improve code quality.

Summary:

This PR updates the README.md to include optional HTTPS certificate and private key details under the Requirements section, enhancing clarity for users.

Key points:

You can configure Ellipsis to address comments with a direct commit or a side PR, see docs.

Something look wrong?: If this Pull Request doesn't address the comments left on the above pull request, create a new PR review with more details. For more information, check the documentation.

Generated with :heart: by ellipsis.dev

melvincarvalho commented 2 months ago

@elipsis-dev no need to escape code fences

ellipsis-dev[bot] commented 2 months ago

@melvincarvalho, I have addressed your comments in commit https://github.com/nosdav/server/pull/15/commits/5e1baf0483bbdcfa27df1527ff702654c6354782

melvincarvalho commented 2 months ago

The Requirements heading is there twice it should only be there once. Remove the extra line @elipsis-dev

ellipsis-dev[bot] commented 2 months ago

@melvincarvalho, I have addressed your comments in commit https://github.com/nosdav/server/pull/15/commits/8734a1d22fb8408abaf1e0ef6058e0833f07220a