-
Notifications
You must be signed in to change notification settings - Fork 196
[core] pnpm migration #1594
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
[core] pnpm migration #1594
Conversation
|
maybe add a new workflow file that runs the new build/tests? |
@mattpodwysocki Love this work! Would be great to have it in - @pelikhan tells me it solves many of the things I've been asking for 😄 |
…naiscript into feat/pnpm-migration
…naiscript into feat/pnpm-migration
…a with core functions
…m compile:action'
Investigator reportContext collection
AI Analysis
|
…in package management
Linter: variable-namesNo test cases found for the newly added Linter: statsHere are some basic stats on the provided diff:
Linter: no-fix-mesNo "FIXME" comments detected, all good here. Linter: grumpy-dev
Linter: diagramsHere’s a summary of the workflow changes in diagram format: graph LR
subgraph ".github/workflows"
old[Old Steps]
new[Updated Steps]
old -->|Files Modified| anthropic.yml
old -->|Files Modified| azure.yml
old -->|Files Modified| build-genai-commit.yml
old -->|Files Modified| build-genai.yml
old -->|Files Modified| build.yml
old -->|Files Modified| cli-test.yml
old -->|Files Modified| docs.yml
old -->|Files Modified| genai-azure-service-principal.yml
old -->|Files Modified| genai-blog-post.yml
old -->|Files Modified| genai-commander.yml
old -->|Files Modified| genai-commit-tests.yml
old -->|Files Modified| genai-docs.yml
old -->|Files Modified| genai-iat.yml
old -->|Files Modified| genai-investigator.yml
old -->|Files Modified| genai-issue-bot.yml
old -->|Files Modified| genai-issue-review.yml
old -->|Files Modified| genai-linters.yml
old -->|Files Modified| genai-pr-commit-review.yml
old -->|Files Modified| genai-pr-docs-commit-review.yml
new[.editorconfig Added]
end
Linter: descriptionThe description of the intended purpose of these changes is missing or unclear. Provide a clear justification for:
Linter: comments
|
- Updated package.json scripts to use pnpm instead of yarn for genai commands. - Modified README updater documentation to reflect the change from yarn to pnpm. - Changed comments in show_n_tell.genai.mjs to use pnpm commands. - Updated vscode package.json to use pnpm for prepublish script. - Removed force install script from slides package.json.
…move obsolete package.json in API
This is a refactoring to introduce the following:
@genaiscript/core
@genaiscript/runtime