Skip to content

[Chore] Add CI workflow, pre-commit checks, and test requirements#7

Merged
jiahy0825 merged 1 commit intoSandAI-org:mainfrom
cennn:chore/add-ci-workflow
Mar 28, 2026
Merged

[Chore] Add CI workflow, pre-commit checks, and test requirements#7
jiahy0825 merged 1 commit intoSandAI-org:mainfrom
cennn:chore/add-ci-workflow

Conversation

@cennn
Copy link
Copy Markdown
Collaborator

@cennn cennn commented Mar 28, 2026

🗂️ PR Category

  • 🧹 Chore (Dependencies, CI/CD, Configuration, etc.)
  • 🧪 Testing

📝 Description

Add self-hosted CI for MagiCompiler (triggered on PR to main).

  • integration_test.yml: Chinese character check, pre-commit code style, MagiCompiler install & pytest -v tests/.
  • check_chinese_chars.py: Diff-based Chinese character detection.
  • requirements-test.txt: Pinned test-only dependencies (timm, diffusers, transformers).
  • Fix formatting issues in Dockerfile.magi_compiler.base and requirements.txt (caught by pre-commit).

@cennn cennn force-pushed the chore/add-ci-workflow branch 5 times, most recently from 6f1390b to fdaeb73 Compare March 28, 2026 07:41
@cennn cennn force-pushed the chore/add-ci-workflow branch from fdaeb73 to b13eb1e Compare March 28, 2026 07:51
@cennn cennn changed the title [Dev] Add CI workflow and test requirements [Chore] Add CI workflow, pre-commit checks, and test requirements Mar 28, 2026
Copy link
Copy Markdown
Collaborator

@jiahy0825 jiahy0825 left a comment

Choose a reason for hiding this comment

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

LGTM

@jiahy0825 jiahy0825 merged commit 04cd2e7 into SandAI-org:main Mar 28, 2026
3 checks passed
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.

2 participants