Skip to content

Cleanup JS Publisher Status Channel#1993

Merged
piotrpio merged 1 commit intonats-io:mainfrom
samuelattwood:sattwood/cleanup-statusch
Jan 21, 2026
Merged

Cleanup JS Publisher Status Channel#1993
piotrpio merged 1 commit intonats-io:mainfrom
samuelattwood:sattwood/cleanup-statusch

Conversation

@samuelattwood
Copy link
Copy Markdown
Member

  • Remove channel from conn map rather than just closing it

Copy link
Copy Markdown

@sethjback sethjback left a comment

Choose a reason for hiding this comment

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

LGTM!

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 85.325% (+0.1%) from 85.225%
when pulling 1512940 on samuelattwood:sattwood/cleanup-statusch
into 69d671a on nats-io:main.

Copy link
Copy Markdown
Collaborator

@piotrpio piotrpio left a comment

Choose a reason for hiding this comment

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

LGTM!

@piotrpio piotrpio merged commit 5581079 into nats-io:main Jan 21, 2026
4 checks passed
@piotrpio piotrpio mentioned this pull request Feb 23, 2026
@piotrpio piotrpio mentioned this pull request Mar 25, 2026
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.

4 participants