Conversation
Deploying repid with
|
| Latest commit: |
3079b0d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e678e646.repid.pages.dev |
| Branch Preview URL: | https://nats.repid.pages.dev |
Coverage Report
98 files skipped due to complete coverage. |
There was a problem hiding this comment.
Pull request overview
Adds NATS (JetStream) as a supported message broker connection in repid, along with integration test coverage and CI/test-container wiring.
Changes:
- Introduce
NatsServer,NatsSubscriber, andNatsReceivedMessageimplementations for publish/subscribe, ack/nack/reject, and reply. - Add a NATS Docker test container +
nats_connectionfixture and include it in the integration test matrix. - Add NATS-specific integration tests and an optional dependency group for
nats-py.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
repid/connections/nats/message_broker.py |
New NATS/JetStream server + subscriber + received-message implementation. |
repid/connections/nats/__init__.py |
Export NatsServer from the new NATS connection package. |
repid/connections/__init__.py |
Conditionally expose NatsServer when nats is installed. |
tests/integration/conftest.py |
Add NATS container + nats_connection fixture and include it in the parametrized integration setup. |
tests/integration/test_nats_specific.py |
Add integration tests covering NATS publish/subscribe and edge cases. |
pyproject.toml |
Add nats optional dependency (nats-py). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
aleksul
force-pushed
the
nats
branch
2 times, most recently
from
c953c84 to
f278ade
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
aleksul
force-pushed
the
nats
branch
3 times, most recently
from
89703e2 to
adef69f
Compare
No description provided.