-
Notifications
You must be signed in to change notification settings - Fork 330
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
chore: organize ts files and tooling #466
Conversation
✅ Heimdall Review Status
|
023bba4
to
59e890f
Compare
59e890f
to
1d2adf4
Compare
@@ -15,7 +15,8 @@ jobs: | |||
with: | |||
node-version: "18" | |||
registry-url: "https://registry.npmjs.org" | |||
- run: npm ci |
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.
npm ci
is ran in the following step
# Install dependencies in parent directory first | ||
- run: npm install | ||
# formatting/lint in working directory | ||
cache-dependency-path: ./typescript |
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.
Code wise, happy with the refactor. Makes sense.
Readme update wise, your changes are clear and make sense.
Functionality wise, can confirm:
- Root does nothing, no commands work from root
- From new root (/typescript), every core turbo command (build, format/format:check, lint/lint:fix) is still being ran on all 8 node packages
- Build still injects
@coinbase/agentkit
into the chatbot examples properly for good devx
Overall, LGTM
What changed?
Moves all TypeScript ecosystem files & tooling into the root
typescript/
folder for better organization & separation between TypeScript and Python, and in preparation for the incoming addition of ecosystem-specific changelog management tooling.In general, we are now treating the
coinbase/agentkit
repo as a repo that houses two distinct monorepos: one for TypeScript and one for Python.All relevant readmes and contributing files have been updated, along with GitHub Actions.