-
Notifications
You must be signed in to change notification settings - Fork 25
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: improve radashi maintenance/ci scripts #315
Conversation
@aleclarson With this PR, the I mention this because, currently, when trying to use the command, we get this error: Regarding this, I can think of three ways to address the issue:
|
@MarlonPassos-git Oops, thanks for the report! I've made the necessary changes in |
A stable release To install: pnpm add [email protected] |
Summary
node --experimental-strip-types
instead of tsx. Otherwise, tsx is used like before.scripts/run.js
module), so we can remove fragilepnpm install
calls from GitHub workflows.test-single
,test-branch
, andcheckout-pr
scripts have been rewritten in Node.js for better maintainability.scripts
share a single lockfile (by using the--lockfile-dir
option ofpnpm install
).pnpm radashi import 123
command (see radashi-helper for details).Remaining tasks
CONTRIBUTING.md
README.md
Related issue, if any:
Closes #311
In #269, issues with Bash were reported: