Skip to content

Comments

Consolidate streaming-related modules under proxystore.stream#468

Merged
gpauloski merged 1 commit intomainfrom
issue-463
Jan 29, 2024
Merged

Consolidate streaming-related modules under proxystore.stream#468
gpauloski merged 1 commit intomainfrom
issue-463

Conversation

@gpauloski
Copy link
Collaborator

Description

Makes proxystore.stream a subpackage containing the main streaming interface and the publisher/subscriber shims that used to be in the top-level proxystore.pubsub subpackage.

Fixes

Type of Change

  • Breaking Change (fix or enhancement which changes existing semantics of the public interface)
  • Enhancement (new features or improvements to existing functionality)
  • Bug (fixes for a bug or issue)
  • Internal (refactoring, style changes, testing, optimizations)
  • Documentation update (changes to documentation or examples)
  • Package (dependencies, versions, package metadata)
  • Development (CI workflows, pre-commit, linters, templates)
  • Security (security related changes)

Testing

Existing tests pass as usual. No changes to functionality besides import paths.

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Tags added to PR (e.g., breaking, bug, enhancement, internal, documentation, package, development, security).
  • Code changes pass pre-commit (e.g., mypy, ruff, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

@gpauloski gpauloski added enhancement New features or improvements to existing functionality internal Refactoring, style changes, testing, or code optimizations labels Jan 29, 2024
@gpauloski gpauloski merged commit 080f2e3 into main Jan 29, 2024
@gpauloski gpauloski deleted the issue-463 branch January 29, 2024 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or improvements to existing functionality internal Refactoring, style changes, testing, or code optimizations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move proxystore.pubsub to proxystore.stream.shims

1 participant