Skip to content

Commit

Permalink
fix: release pipeline 3 (#784)
Browse files Browse the repository at this point in the history
- Remove accidental dry-run from jsr publish
- Fix cross dockerfile

#### Migration notes

...

- [ ] The change comes with new or modified tests
- [ ] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change
  • Loading branch information
Yohe-Am authored Jul 7, 2024
1 parent fb240f0 commit 7765a90
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
1 change: 1 addition & 0 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ jobs:
- uses: ncipollo/release-action@v1
with:
tag: ${{ steps.latest-tag.outputs.tag }}
allowUpdates: ${{ github.event_name == 'workflow_dispatch' }}
makeLatest: true
bodyFile: "CHANGE.md"
discussionCategory: "Announcements"
Expand Down
6 changes: 4 additions & 2 deletions dev/cross.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,10 @@ RUN set -eux \

ENV GHJK_SHARE_DIR=/ghjk
ARG GHJK_VERSION=0.2.0
RUN GHJK_INSTALL_EXE_DIR=/usr/bin GHJK_INSTALL_HOOK_SHELLS=bash \
deno run -A https://raw.github.com/metatypedev/ghjk/$GHJK_VERSION/install.ts
RUN curl -fsSL https://raw.github.com/metatypedev/ghjk/$GHJK_VERSION/install.sh \
| GHJK_INSTALL_EXE_DIR=/usr/bin GHJK_INSTALL_HOOK_SHELLS=bash sh

WORKDIR /app

COPY dev/*.ts dev/
COPY ghjk.ts .
Expand Down
2 changes: 1 addition & 1 deletion dev/tasks-build.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ const tasks: Record<string, DenoTaskDefArgs> = {
"build-jsr-pub": {
async fn($) {
await $`deno run -A typegraph/deno/dev/jsr-gen.ts`;
await $`cd typegraph/deno/sdk && deno publish --dry-run --allow-slow-types --allow-dirty`;
await $`cd typegraph/deno/sdk && deno publish --allow-slow-types --allow-dirty`;
},
},
"build-tgraph-py": {
Expand Down

0 comments on commit 7765a90

Please sign in to comment.