Andrew Powell
Andrew Powell
@CharlieHelps this is a pretty poor attempt. You ignored instructions in .charlie and you haven't followed the playbook in .charlie. reset to master and try again.
@CharlieHelps even after your reset you missed the pnpm lockfile issue. run `pnpm i` and commit the result. DO NOTHING OTHER THAN THAT to resolve the CI failure. THERE IS...
@CharlieHelps ``` RUN v4.0.2 /home/runner/work/plugins/plugins/packages/auto-install stdout | test/pnpm.test.ts > pnpm installing node-noop... stdout | test/npm.test.ts > npm installing node-noop... stdout | test/npm-bare.test.ts > npm, bare installing node-noop... ✓ test/pnpm.test.ts (1...
@CharlieHelps there are four failing tests in CI
@CharlieHelps you have failing tests. please reference the `alias` package's setup from the `master` branch
@CharlieHelps it's getting pretty tiresome that you claim everything is working but your commits result in another CI failure: ✓ test/pnpm-bare.test.ts > pnpm, bare 1008ms × test/npm.test.ts > npm 5012ms...
As always thanks for the PR. I'm not going to approve this, however. It's a micro-optimization at the expense of readability with no clear advantage that any typical user will...
checking in two years later to say that this is still sorely needed
@altinokdarici looks like there's a prettier issue. please run lint and tag me again when you update