Skip to content

feat(ci): target desktop releases to build commit - #13

Merged
lxrzlyr merged 1 commit into
devfrom
ci-release-target-dev
Jun 30, 2026
Merged

feat(ci): target desktop releases to build commit#13
lxrzlyr merged 1 commit into
devfrom
ci-release-target-dev

Conversation

@lxrzlyr

@lxrzlyr lxrzlyr commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Issue for this PR

N/A - release automation correction.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Targets automatic desktop Release tags at the main build commit SHA instead of the repository default branch. This keeps app-v*-main.* releases tied to the exact commit that produced the packages.

How did you verify your code works?

  • Ran actionlint .github/workflows/desktop-build.yml
  • Push hook ran bun turbo typecheck successfully: 23/23 tasks

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@lxrzlyr
lxrzlyr merged commit 6db77ad into dev Jun 30, 2026
4 of 10 checks passed
@lxrzlyr
lxrzlyr deleted the ci-release-target-dev branch June 30, 2026 11:21
lxrzlyr pushed a commit that referenced this pull request Jul 3, 2026
### Issue for this PR

N/A - release automation correction.

### Type of change

- [x] Bug fix
- [ ] New feature
- [ ] Refactor / code improvement
- [ ] Documentation

### What does this PR do?

Targets automatic desktop Release tags at the main build commit SHA
instead of the repository default branch. This keeps `app-v*-main.*`
releases tied to the exact commit that produced the packages.

### How did you verify your code works?

- Ran `actionlint .github/workflows/desktop-build.yml`
- Push hook ran `bun turbo typecheck` successfully: 23/23 tasks

### Screenshots / recordings

_If this is a UI change, please include a screenshot or recording._

### Checklist

- [x] I have tested my changes locally
- [x] I have not included unrelated changes in this PR
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