Skip to content

Add local RPG-Kit template source and managed venv setup#55

Merged
QingtaoLi1 merged 6 commits into
mainfrom
gitignore
May 19, 2026
Merged

Add local RPG-Kit template source and managed venv setup#55
QingtaoLi1 merged 6 commits into
mainfrom
gitignore

Conversation

@QingtaoLi1
Copy link
Copy Markdown
Contributor

  • Add --source support for rpgkit init and rpgkit update, allowing templates to be built from a local RPG-Kit checkout instead of downloading a GitHub release asset.
  • Automatically create/update .venv_rpgkit during init and update, install dependencies from .rpgkit/pyproject.toml, ignore the venv in generated .gitignore, and show activation instructions after setup.
  • Narrow the pre-release workflow branch trigger。
  • Update README news.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for building RPG-Kit templates from a local checkout (instead of downloading release assets) and introduces an automated “managed” Python venv (.venv_rpgkit) setup during rpgkit init/update, alongside some small repo/workflow/documentation updates.

Changes:

  • Add --source to rpgkit init / rpgkit update and wire it to build a local template archive via the release-packaging scripts.
  • Add .venv_rpgkit management: create/update the venv, install Python deps, ignore it via the managed .gitignore, and print activation instructions.
  • Tighten the rpgkit-pre-release workflow branch trigger and refresh README/news text.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
RPG-Kit/src/rpgkit_cli/__init__.py Adds --source flow for local template packaging, adds .venv_rpgkit setup + related messaging, and updates ignore/prune lists.
README.md Updates badge labels and refreshes the News section.
.gitignore Ignores workspace/ at repo root.
.github/workflows/scripts/rpgkit/create-release-packages.sh Allows overriding Python interpreter and copies pyproject.toml into generated .rpgkit/.
.github/workflows/scripts/rpgkit/create-release-packages.ps1 Copies pyproject.toml into generated .rpgkit/ (PowerShell variant).
.github/workflows/rpgkit-pre-release.yml Narrows push trigger from dev + dev/** to dev only.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +2812 to +2815
result = subprocess.run(
["bash", str(release_script), version],
cwd=repo_root,
env=env,
Comment thread RPG-Kit/src/rpgkit_cli/__init__.py
Comment on lines +3692 to +3696
steps_lines.append(
f"{step_num}. Activate the RPG-Kit Python environment: "
f"[cyan]source .venv_rpgkit/bin/activate[/cyan]"
)
step_num += 1
Comment thread RPG-Kit/src/rpgkit_cli/__init__.py Outdated
Comment on lines +4039 to +4047
console.print(
Panel(
"Activate the RPG-Kit Python environment before using slash commands:\n\n"
"[cyan]source .venv_rpgkit/bin/activate[/cyan]",
title="[yellow]Environment Setup[/yellow]",
border_style="yellow",
padding=(1, 2),
)
)
Comment on lines +118 to +119
[[ -f pyproject.toml ]] && { cp pyproject.toml "$SPEC_DIR/pyproject.toml"; echo "Copied pyproject.toml -> .rpgkit"; }

Comment on lines +3297 to +3304
source: Optional[Path] = typer.Option(
None,
"--source",
help=(
"Use a local RPG-Kit source checkout to build and install the "
"template package instead of downloading a release asset."
),
),
Comment on lines +159 to +163
if (Test-Path "pyproject.toml") {
Copy-Item -Path "pyproject.toml" -Destination (Join-Path $specDir "pyproject.toml") -Force
Write-Host "Copied pyproject.toml -> .rpgkit"
}

@QingtaoLi1 QingtaoLi1 merged commit 4ae199d into main May 19, 2026
2 checks passed
@QingtaoLi1 QingtaoLi1 deleted the gitignore branch May 19, 2026 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants