Skip to content

Refresh post-checkout and shell_hook.sh scripts on luca update#55

Merged
albertodebortoli merged 1 commit intomainfrom
refresh-scripts-on-update
Apr 8, 2026
Merged

Refresh post-checkout and shell_hook.sh scripts on luca update#55
albertodebortoli merged 1 commit intomainfrom
refresh-scripts-on-update

Conversation

@albertodebortoli
Copy link
Copy Markdown
Member

@albertodebortoli albertodebortoli commented Apr 8, 2026

Description

  • SelfUpdater now downloads the latest post-checkout and shell_hook.sh from GitHub to ~/.luca/ on every luca update.
  • Scripts are refreshed even when the binary is already up to date.
  • Failures are non-fatal — errors are logged as warnings without interrupting the update.
  • Depends on Detect and replace outdated Luca-managed post-checkout hooks #54.

Type of Change

  • Feature

How Has This Been Tested?

  • Added / updated unit tests
  • Tested on macOS (arch: arm64)

Checklist

  • Swift code builds locally (swift build)
  • Tests pass locally (swift test)
  • Code style / formatting respected

CI Considerations

  • Affects build time notably
  • Requires new secrets / env vars
  • Alters release process

Breaking Changes?

  • No

@albertodebortoli
Refresh post-checkout and shell_hook.sh scripts on luca update
75cad83 
SelfUpdater now downloads the latest post-checkout and shell_hook.sh
from GitHub to ~/.luca/ on every luca update, even when the binary
is already up to date. Failures are non-fatal.
@albertodebortoli albertodebortoli added this to the 0.14.0 milestone Apr 8, 2026
@albertodebortoli albertodebortoli added the feature New feature or enhancement label Apr 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ources/LucaCore/Core/SelfUpdater/SelfUpdater.swift 95.23% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@albertodebortoli albertodebortoli marked this pull request as ready for review April 8, 2026 22:47
Base automatically changed from detect-replace-outdated-luca-hooks to main April 8, 2026 22:52
@albertodebortoli albertodebortoli merged commit 8b80b4a into main Apr 8, 2026
5 checks passed
@albertodebortoli albertodebortoli deleted the refresh-scripts-on-update branch April 8, 2026 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature New feature or enhancement

Projects

None yet

Milestone

0.14.0

Development

Successfully merging this pull request may close these issues.

1 participant

@albertodebortoli