-
Notifications
You must be signed in to change notification settings - Fork 126
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.
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
refactor(provider-bindgen): fix import & export semantics #1435
refactor(provider-bindgen): fix import & export semantics #1435
Conversation
The current semantics are unintuitive -- interfaces "exported" from a provider signifies functionality *provided* to the lattice, and the WIT worlds should reflect that. At present, functionality exposed to the lattice for actors to use is signified by WIT *imports* in the provider worlds. This commit reverses the import and export semantics that were used for WIT contracts in the providers, to make them more intuitive. Signed-off-by: Victor Adossi <vadossi@cosmonic.com>
e3ccc7a
to
119bb2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really awesome stuff. I love the wits and some of the new config code, might warrant a second review from another maintainer more familiar with provider SDK but the semantics are great
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really awesome stuff. I love the wits and some of the new config code, might warrant a second review from another maintainer more familiar with provider SDK but the semantics are great
@vados-cosmonic can you point to where the logical changes happened here, or split into multiple commits? With this being only one commit, I can't easily tell what was simply split into multiple files, and where the import/export semantics were reversed |
Sure @connorsmith256 -- noted some areas that saw change -- most of the rest was splitting up the functionality |
Feature or Problem
The current semantics are unintuitive -- interfaces "exported" from a provider signifies functionality provided to the lattice, and the WIT worlds should reflect that.
At present, functionality exposed to the lattice for actors to use is signified by WIT imports in the provider worlds.
This commit reverses the import and export semantics that were used for WIT contracts in the providers, to make them more intuitive.
Resolves #715
Related Issues
Release Information
Consumer Impact
Testing
Unit Test(s)
Acceptance or Integration
Manual Verification