Skip to content

[WIP] Add option to merge nats core and jetstream subscriptions #971

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
AdamPayzant wants to merge 5 commits into
base: main
Choose a base branch
from

Addressing comments, removing redundant handling

7655acd
Select commit
Loading
Failed to load commit list.
Draft

[WIP] Add option to merge nats core and jetstream subscriptions #971

Addressing comments, removing redundant handling
7655acd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 17, 2026 in 1s

56.25% of diff hit (within 20.00% threshold of 70.27%)

View this Pull Request on Codecov

56.25% of diff hit (within 20.00% threshold of 70.27%)

Annotations

Check warning on line 1430 in src/sub.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/sub.c#L1429-L1430 

Added lines #L1429 - L1430 were not covered by tests

Check warning on line 1439 in src/sub.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/sub.c#L1439 

Added line #L1439 was not covered by tests

Check warning on line 1459 in src/sub.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/sub.c#L1459 

Added line #L1459 was not covered by tests

Check warning on line 1473 in src/sub.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/sub.c#L1473 

Added line #L1473 was not covered by tests

Check warning on line 1494 in src/sub.c

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/sub.c#L1489-L1494 

Added lines #L1489 - L1494 were not covered by tests
View more details on Codecov