Skip to content

Add lint CI workflow#192

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/package-tokenizers-swift-pm
Draft

Add lint CI workflow#192
Copilot wants to merge 4 commits into
mainfrom
copilot/package-tokenizers-swift-pm

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 26, 2026

The repo has lintrunner fully configured (.lintrunner.toml with FLAKE8, UFMT, CLANGFORMAT, CMAKE, NEWLINE, MYPY) but no CI job was running it.

  • Adds .github/workflows/lint.yml triggered on PRs and pushes to main
  • Checks out with full history for merge_base_with support
  • Installs deps from requirements-lintrunner.txt, runs lintrunner init, then lintrunner --all-files

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 26, 2026
Copilot AI linked an issue May 26, 2026 that may be closed by this pull request
Copilot AI added 2 commits May 26, 2026 23:51
Create Package.swift manifest that compiles all C++ sources (tokenizers core,
abseil-cpp, re2, sentencepiece with bundled protobuf-lite, nlohmann/json headers,
llama.cpp-unicode) into a single Swift PM library target.

Includes:
- Package.swift with explicit source file lists and cxxSettings
- swift-config/config.h for sentencepiece (replaces CMake-generated config)
- .gitignore entries for Swift PM build artifacts
Copilot AI changed the title [WIP] Package tokenizers for integration with ExecuTorch Add Swift Package Manager support May 26, 2026
Copilot AI requested a review from kirklandsign May 26, 2026 23:52
Copilot AI changed the title Add Swift Package Manager support Add lint CI workflow May 27, 2026
Comment on lines +16 to +35
name: lintrunner
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: '3.12'

- name: Install lintrunner dependencies
run: pip install -r requirements-lintrunner.txt

- name: Initialize lintrunner
run: lintrunner init

- name: Run lintrunner
run: lintrunner --force-color --all-files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package tokenizers into swift PM

3 participants