Skip to content
This repository was archived by the owner on Jan 24, 2026. It is now read-only.

Fix tokenizers-sys to use HTTPS like other git clones#70

Merged
bpkeene merged 1 commit intoargmaxinc:mainfrom
agg23:tokenizers-sys-clone-fix
Jun 24, 2025
Merged

Fix tokenizers-sys to use HTTPS like other git clones#70
bpkeene merged 1 commit intoargmaxinc:mainfrom
agg23:tokenizers-sys-clone-fix

Conversation

@agg23
Copy link
Contributor

@agg23 agg23 commented Jun 19, 2025

Description

For some reason the build script use SSH to clone the tokenizers-sys repo, whereas all other clones use HTTPS. This will break on environments where SSH is not configured. Switch to HTTPS to match the other repos.

Type of Change

  • Bug fix 🐛
  • New feature 🚀
  • Refactor 🔄
  • Documentation update 📖
  • Other (please describe)

Test Plan

  • I have run bash test/test_build_all.sh and it ran successfully
  • I have tested this change on all relevant platforms.

Checklist

  • My code follows the project's style guidelines.
  • I have updated relevant documentation (if applicable).
  • I have added appropriate tests (if applicable).
  • I have self-reviewed my code before requesting review.

Copy link
Contributor

@keith4ever keith4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, approved.

Copy link
Contributor

@dylanangus dylanangus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bpkeene bpkeene merged commit 2881928 into argmaxinc:main Jun 24, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants