Skip to content

Add vapour publish workflow with required flags - #1

Closed
madhavkhoslaa wants to merge 1 commit into
mainfrom
feat/publish-required-flags
Closed

madhavkhoslaa wants to merge 1 commit into
mainfrom
feat/publish-required-flags

Conversation

@madhavkhoslaa

Copy link
Copy Markdown

Summary

  • add .github/workflows/vapour-publish.yml that builds web output and calls vapour-computer/vapour-ci@v1
  • pass required publish fields:
    • version from tag name with leading v stripped
    • notes text

Why

Requested in PUBLISH-FLAGS-TASK so the tutorial repo has a publish workflow compatible with required publish flags.

Drafted with opencode (openai/gpt-5.3-codex).

@madhavkhoslaa

Copy link
Copy Markdown
Author

Closing this PR per review: it targets the wrong base () and its workflow writes build output into . Replaced by a new PR from that keeps /assets Cargo.lock flake.lock LICENSE README.md
build Cargo.toml flake.nix mobile src
build.rs credits index.html nix Trunk.toml and only adds required version+notes inputs.

@madhavkhoslaa

Copy link
Copy Markdown
Author

Follow-up note: this was closed because it targeted main and changed build output to build/web.
Replacement PR uses base vapour-tutorial, keeps trunk build --dist dist and dir: ., and only adds required version/notes wiring.

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.

1 participant