Skip to content

Add CI to verify that the library is buildable and correct on style and format - #34

Merged
Kyle9410-Chen merged 7 commits into
mainfrom
ci/add-ci-testing
Aug 11, 2026
Merged

Add CI to verify that the library is buildable and correct on style and format#34
Kyle9410-Chen merged 7 commits into
mainfrom
ci/add-ci-testing

Conversation

@yukicoder0509

@yukicoder0509 yukicoder0509 commented Jul 28, 2026

Copy link
Copy Markdown
Member

Change type

  • CI

Purpose

  • Add CI to test the following standards
    • Can build: go build ./...
    • Pass Go lint check: golangci-lint run
    • Pass all test cases: go test ./... -race -cover
    • Can use CLI tool to init a project
  • Remove unused function in pkg/log/logger.go

@yukicoder0509
yukicoder0509 requested review from Kyle9410-Chen, howardyu60211 and ilsao and removed request for howardyu60211 July 28, 2026 15:25
Comment thread .github/workflows/CI.yaml Outdated
@yukicoder0509
yukicoder0509 requested a review from ilsao July 29, 2026 07:12
@Kyle9410-Chen
Kyle9410-Chen merged commit d7f5913 into main Aug 11, 2026
4 checks passed
@Kyle9410-Chen
Kyle9410-Chen deleted the ci/add-ci-testing branch August 11, 2026 14:24
@YukinaMochizuki YukinaMochizuki mentioned this pull request Aug 27, 2026
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.

3 participants