Skip to content
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

Fix subscription topic handling #323

Merged
merged 3 commits into from
Oct 30, 2023

SubscriptionTopic: constrain constructor

4e54204
Select commit
Loading
Failed to load commit list.
Merged

Fix subscription topic handling #323

SubscriptionTopic: constrain constructor
4e54204
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Oct 30, 2023 in 1s

46.93% of diff hit (target 56.08%)

View this Pull Request on Codecov

46.93% of diff hit (target 56.08%)

Annotations

Check warning on line 99 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L99 

Added line #L99 was not covered by tests

Check warning on line 125 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L125 

Added line #L125 was not covered by tests

Check warning on line 132 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L132 

Added line #L132 was not covered by tests

Check warning on line 137 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L137 

Added line #L137 was not covered by tests

Check warning on line 140 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L140 

Added line #L140 was not covered by tests

Check warning on line 151 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L150-L151 

Added lines #L150 - L151 were not covered by tests

Check warning on line 156 in src/core/include/SubscriptionTopic.hpp

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/core/include/SubscriptionTopic.hpp#L155-L156 

Added lines #L155 - L156 were not covered by tests
View more details on Codecov