Skip to content

BREAKING: Remove wildcard imports for ETSI/ IEEE standards - #15

Merged
jbeyerstedt merged 2 commits into
mainfrom
breaking/remove-wildcard-imports
Jul 21, 2026
Merged

BREAKING: Remove wildcard imports for ETSI/ IEEE standards#15
jbeyerstedt merged 2 commits into
mainfrom
breaking/remove-wildcard-imports

Conversation

@jbeyerstedt

Copy link
Copy Markdown
Collaborator

PR #14 has refactored the message definitions to be one file per referenced standard, but kept the public interface the same by using a wildcard "re-export" of the message types.

But this clutters the docs quite a lot and it would be nicer to have the "external" definitions fully contained in their own modules. Therefore this PR removes the wildcard "use" statements and updates all usages of the types.

Users of this crate need to add en302636_4_1 or ieee1609dot2 to their type qualifiers (or use a wildcard import).

@jbeyerstedt jbeyerstedt added this to the v0.4.0 milestone Apr 20, 2026
@jbeyerstedt
jbeyerstedt force-pushed the breaking/remove-wildcard-imports branch from eaa8368 to 896d57a Compare April 20, 2026 15:26
@jbeyerstedt
jbeyerstedt force-pushed the breaking/remove-wildcard-imports branch from 896d57a to 7dbda2f Compare May 12, 2026 13:18
@jbeyerstedt
jbeyerstedt force-pushed the breaking/remove-wildcard-imports branch 4 times, most recently from aa209cf to 23e572e Compare July 16, 2026 07:03
@jbeyerstedt
jbeyerstedt force-pushed the breaking/remove-wildcard-imports branch from 23e572e to 26bf582 Compare July 21, 2026 09:05

@BendixBuchheister BendixBuchheister left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@jbeyerstedt
jbeyerstedt merged commit 488171d into main Jul 21, 2026
3 checks passed
@jbeyerstedt
jbeyerstedt deleted the breaking/remove-wildcard-imports branch July 21, 2026 12:53
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