Skip to content

Commit bf00974

Browse files
ncrmroclaude
andcommitted
fix: remove nix result symlink and add dry-run build to release workflow
- Remove accidentally committed `result` symlink that pointed to nix store - Add `result` to .gitignore to prevent future accidental commits - Add PR trigger for release.yml changes to catch build issues early Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 11ffd31 commit bf00974

3 files changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/release.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,10 @@ name: Release to PyPI
33
on:
44
release:
55
types: [published]
6+
# Dry-run build on PRs to catch build issues before release
7+
pull_request:
8+
paths:
9+
- ".github/workflows/release.yml"
610

711
jobs:
812
build:
@@ -37,6 +41,7 @@ jobs:
3741

3842
publish:
3943
needs: build
44+
if: github.event_name == 'release'
4045
runs-on: ubuntu-latest
4146
environment: pypi
4247
permissions:
@@ -53,6 +58,7 @@ jobs:
5358

5459
update-homebrew:
5560
needs: publish
61+
if: github.event_name == 'release'
5662
runs-on: ubuntu-latest
5763
steps:
5864
- name: Trigger Homebrew formula update

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,9 @@ dmypy.json
6969
# direnv
7070
.direnv/
7171

72+
# Nix build output
73+
result
74+
7275
# Jupyter Notebook
7376
.ipynb_checkpoints
7477

result

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)