Skip to content

Subscriber to server - #241

Merged
FrancescAlted merged 4 commits into
mainfrom
subscriber-to-server
Oct 1, 2025
Merged

Subscriber to server#241
FrancescAlted merged 4 commits into
mainfrom
subscriber-to-server

Conversation

@FrancescAlted

Copy link
Copy Markdown
Member

Most of the references to the previous publish-subscriber are gone now, and replaced by client-server paradigm. We still need to decide what to do with the independent-services.md tutorial.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames terminology throughout the codebase, replacing "subscriber" with "server" and "publisher" with "client" to reflect a client-server paradigm instead of the previous publish-subscribe architecture.

  • Updated references from "subscriber" to "server" in comments, documentation, and configuration files
  • Changed "publisher" references to "client" throughout documentation and specifications
  • Maintained code functionality while updating terminology for clarity

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Updated comment about HDF5 support from "publisher" to "client"
examples/query_server.py Changed comments referencing "subscriber" to "server"
doc/tutorials/launching-services.md Updated tutorial text replacing broker/publisher/subscriber with client/server terminology
doc/tutorials/independent-services.md Removed broker references and updated publisher/subscriber to client/server
doc/tutorials/configuration.md Updated configuration examples from "subscriber" to "server"
doc/tutorials/cli.md Changed references from "subscriber" to "server" in CLI documentation
caterva2/tests/services.py Updated module docstring from "subscriber" to "server"
caterva2/services/settings.py Changed configuration comment from "subscriber" to "server"
caterva2/clients/cli.py Updated help text from "subscriber" to "server"
caterva2/client.py Updated docstrings and comments replacing "subscriber" with "server"
caterva2.sample.toml Updated configuration comments from "subscriber" to "server"
SPECS.md Major documentation update replacing publish-subscribe with client-server terminology
README.md Updated installation and usage instructions with new terminology
README-DEVELOPERS.md Changed reference from "subscriber" to "server"

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jdavid jdavid left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@FrancescAlted
FrancescAlted merged commit 903bc78 into main Oct 1, 2025
3 checks passed
@FrancescAlted
FrancescAlted deleted the subscriber-to-server branch October 1, 2025 15:31
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.

3 participants