Skip to content

feat(pnpm-install): add working-directory input#269

Merged
B4nan merged 1 commit intomainfrom
chore/pnpm-install-working-directory
Apr 16, 2026
Merged

feat(pnpm-install): add working-directory input#269
B4nan merged 1 commit intomainfrom
chore/pnpm-install-working-directory

Conversation

@B4nan
Copy link
Copy Markdown
Member

@B4nan B4nan commented Apr 16, 2026

Summary

  • Adds optional working-directory input (defaults to .) to the pnpm-install composite action
  • Allows repos where the pnpm project is in a subdirectory (e.g. impit/impit-node) to use the shared action instead of inlining pnpm/action-setup + manual install

Backwards-compatible — defaults to repo root, no change for existing callers.

🤖 Generated with Claude Code

Allows repos where the pnpm project lives in a subdirectory (e.g.
impit/impit-node) to use the shared action instead of inlining
pnpm/action-setup + manual install.
@github-actions github-actions bot added this to the 138th sprint - Tooling team milestone Apr 16, 2026
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Apr 16, 2026
@B4nan B4nan added the adhoc Ad-hoc unplanned task added during the sprint. label Apr 16, 2026
@B4nan B4nan requested a review from vladfrangu April 16, 2026 10:13
B4nan added a commit to apify/impit that referenced this pull request Apr 16, 2026
Replace all pnpm/action-setup + manual install with the shared
composite action. Docker builds keep npm install -g pnpm (can't
use GH actions inside containers). Depends on apify/workflows#269
for the working-directory input.
@B4nan B4nan merged commit ea5bf00 into main Apr 16, 2026
3 of 5 checks passed
@B4nan B4nan deleted the chore/pnpm-install-working-directory branch April 16, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants