Skip to content

feat: add package-manager environment configuration - #148

Merged
fengmk2 merged 6 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/package-manager-config
Sep 12, 2026
Merged

feat: add package-manager environment configuration#148
fengmk2 merged 6 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/package-manager-config

Conversation

@liangmiQwQ

@liangmiQwQ liangmiQwQ commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Add package-manager for GitHub and GitLab, and packageManager for Azure. Accept a boolean or a mapping of npm, pnpm, yarn, and bun to booleans.

false runs vp env off pm; individual false entries run commands such as vp env off pnpm. True and omitted entries leave the installer default unchanged, without explicit enable commands. Any explicit configuration requires Vite+ 0.3.1+; older versions remain supported when the input is unset.

🤖 Generated with Codex

@liangmiQwQ
liangmiQwQ marked this pull request as ready for review September 12, 2026 05:16
@liangmiQwQ

Copy link
Copy Markdown
Collaborator Author

@fengmk2 I'm ready for both one.

Comment thread README.md
@liangmiQwQ
liangmiQwQ marked this pull request as draft September 12, 2026 08:22
@liangmiQwQ
liangmiQwQ force-pushed the liang/codex/package-manager-config branch from f4bcbb0 to 2aa2a49 Compare September 12, 2026 08:32
@liangmiQwQ
liangmiQwQ marked this pull request as ready for review September 12, 2026 08:34
@liangmiQwQ
liangmiQwQ force-pushed the liang/codex/package-manager-config branch from 2aa2a49 to 6248850 Compare September 12, 2026 09:22
@liangmiQwQ
liangmiQwQ requested a review from fengmk2 September 12, 2026 09:22
@liangmiQwQ
liangmiQwQ marked this pull request as draft September 12, 2026 09:24
@liangmiQwQ
liangmiQwQ marked this pull request as ready for review September 12, 2026 09:28
@fengmk2

fengmk2 commented Sep 12, 2026

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-12T09:44:24.617804Z 6248850 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Comment thread README.md Outdated
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

Reviewed commit: 6248850e80

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@fengmk2 fengmk2 added the run-e2e Request GitLab E2E for the reviewed fork PR commit (requires write access). label Sep 12, 2026
@fengmk2 fengmk2 added run-e2e Request GitLab E2E for the reviewed fork PR commit (requires write access). and removed run-e2e Request GitLab E2E for the reviewed fork PR commit (requires write access). labels Sep 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ GitLab E2E passed

Commit: 3a07037fd3aefd44eeb77554c569e474bab6d0f7
Suite: full · Vite+ version: latest

GitLab pipeline · GitHub Actions run

@fengmk2
fengmk2 merged commit a28dce9 into voidzero-dev:main Sep 12, 2026
64 checks passed
@liangmiQwQ
liangmiQwQ deleted the liang/codex/package-manager-config branch September 12, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e Request GitLab E2E for the reviewed fork PR commit (requires write access).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants