-
Notifications
You must be signed in to change notification settings - Fork 0
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
docs: create README for shared package #64
Conversation
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.
LGTM, just one comment
packages/shared/src/external.ts
Outdated
export { HexUtils as AddressUtils, Caip2Utils, Logger } from "./services/index.js"; | ||
export { InvalidHex as InvalidAddress, InvalidChainId } from "./exceptions/index.js"; |
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.
are these X as Y
correct? i see that in the other packages you're using HexUtils, not AddressUtils
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.
Whoops, refactoring made wrong
packages/shared/README.md
Outdated
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.
💎
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'm just standing on the shoulders of READMEs/repo boilerplate giants
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.
nice 👍🏻
@@ -0,0 +1,68 @@ | |||
# ebo-agent: shared package | |||
|
|||
Shared lib that for EBO agent libraries. |
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.
*Shared lib for EBO agent libraries
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.
Oooh nice catch! 🔭 I'll fix it in the next docs PR
🤖 Linear
Closes GRT-218
Description
shared
packageAddress
service toHexUtils
NormalizedAddress
type toNormalizedHex