Skip to content

Add CI checks to main - #9

Merged
TonyTown6033 merged 2 commits into
mainfrom
migrate/2-ci-checks
Sep 24, 2026
Merged

TonyTown6033 merged 2 commits into
mainfrom
migrate/2-ci-checks

Conversation

@TonyTown6033

Copy link
Copy Markdown
Owner

Summary

  • add a minimal GitHub Actions workflow for Rust checks
  • run formatting, compilation, and tests on pushes and pull requests
  • keep local config.toml ignored so credentials are not committed

Checks

  • cargo fmt --all -- --check
  • cargo check --all-targets
  • cargo test --all-targets

Closes #2

@TonyTown6033

Copy link
Copy Markdown
Owner Author

if compiled it works? where is test case?

@TonyTown6033

Copy link
Copy Markdown
Owner Author

why this pr not include the release for windows and linux

@TonyTown6033

Copy link
Copy Markdown
Owner Author

@codex

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 9f2e33d217

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@TonyTown6033 TonyTown6033 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

next

@TonyTown6033
TonyTown6033 merged commit 71ace48 into main Sep 24, 2026
2 checks passed
@TonyTown6033
TonyTown6033 deleted the migrate/2-ci-checks branch September 24, 2026 11:28
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.

Migrate CI checks to main

2 participants