-
Notifications
You must be signed in to change notification settings - Fork 1
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
[COR-68] Add registry of loaded RPC clients and expose discovery metadata #8
base: master
Are you sure you want to change the base?
Conversation
- Add registry - Update prettier & eslint versions
return client.load(metaPath); | ||
}; | ||
exports.registry = new Registry(); | ||
exports.Registry = Registry; |
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.
Similar to the other PR, this should export WELL_KNOWN_META_PATH
which will be /client
in this case, not /server
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.
Couple of comments.
Most of the change in this PR related to prettier. I think you should update prettier in a PR and get that out first to clean up the diff in this PR. The important changes are lost in all the prettier whitespace changes.
index.js
Outdated
return loadClient; | ||
} | ||
|
||
createWellKnownHandler() { |
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.
I think createWellKnownErrorHandler
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.
Also fix the title. Version bump doesn't describe the changes.
It is about adding a registry of loaded RPC clients and exposing discovery metadata
Version bump to v5..0.0-beta:
Relates to: COR-68