Skip to content

fixed streamer usecase check, enforce source authority non-wildcard#42

Closed
Benedikt-Illich wants to merge 1 commit into
eclipse-uprotocol:mainfrom
ValtechMobility:main
Closed

fixed streamer usecase check, enforce source authority non-wildcard#42
Benedikt-Illich wants to merge 1 commit into
eclipse-uprotocol:mainfrom
ValtechMobility:main

Conversation

@Benedikt-Illich

Copy link
Copy Markdown
Contributor

also adds some logging to the checker function

@PLeVasseur

Copy link
Copy Markdown
Contributor

Opened a separate CI-fix PR here: https://github.com/eclipse-uprotocol/up-transport-vsomeip-rust/pull/43\n\nThis isolates the main CI failures (plus feasible warning cleanups encountered while validating) from #42. The failures are reproducible from and are validated in that independent branch with local clippy/test runs.

@PLeVasseur

Copy link
Copy Markdown
Contributor

Correction: the failures referenced above are reproducible from the upstream main branch.

@PLeVasseur

PLeVasseur commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Clean cross-link: independent CI-fix PR is #43.
The failures addressed there are reproducible from upstream main.

@PLeVasseur

Copy link
Copy Markdown
Contributor

Opened follow-up PR: #44

This carries the same functional change from PR #42 on a clean branch scoped to up-transport-vsomeip/src/determine_message_type.rs, and preserves original author credit via git cherry-pick -x.

@PLeVasseur

Copy link
Copy Markdown
Contributor

Update on follow-up PR #44:

To satisfy current Eclipse Contributor Agreement email checks, the follow-up branch was force-updated to use maintainer commit authorship. Functional scope remains unchanged and still carries the same source patch from this PR.

@PLeVasseur

Copy link
Copy Markdown
Contributor

For the sake of expediency, I fixed this here:
#44

I tried to add you to that PR's commit but it didn't seem to like it for ECA reasons :(

Therefore, closing this one.

@PLeVasseur PLeVasseur closed this Feb 6, 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.

2 participants