Skip to content

Implement architecture validation #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
albertodebortoli wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Implement architecture validation #14

albertodebortoli wants to merge 1 commit into from

Conversation

@albertodebortoli
Copy link
Member

@albertodebortoli albertodebortoli commented Jan 18, 2026

Pull Request Title

This PR adds architecture validation during tool installation to ensure downloaded executables are compatible with the host machine. If an incompatible binary is detected, the installation fails with a clear error message.

Type of Change

  • Feature
  • Bug fix
  • Maintenance / Refactor
  • Documentation
  • CI / Tooling
  • Other (specify)

How Has This Been Tested?

Describe the testing strategy and commands executed.

  • Added / updated unit tests
  • Manually tested locally
  • Tested on macOS (arch: arm64 / x86_64)
  • Other

Screenshots / Demo (if applicable)

Add images, logs, or terminal output.

Checklist

  • Swift code builds locally (swift build)
  • Tests pass locally (swift test)
  • Code style / formatting respected
  • Documentation updated (README / comments)
  • Version / tag alignment considered (if release related)
  • PR title follows conventional style (optional)

CI Considerations

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

Breaking Changes?

  • No
  • Yes

@albertodebortoli albertodebortoli added this to the 0.5.0 milestone Jan 18, 2026
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

None yet

Projects

None yet

Milestone

0.5.0

Development

Successfully merging this pull request may close these issues.

2 participants

@albertodebortoli