Skip to content

[CI] Docker Integration#54

Merged
logic-finder merged 6 commits into
mainfrom
ci/docker-integration
May 20, 2026
Merged

[CI] Docker Integration#54
logic-finder merged 6 commits into
mainfrom
ci/docker-integration

Conversation

@lshqqytiger
Copy link
Copy Markdown
Contributor

@lshqqytiger lshqqytiger commented May 20, 2026

This pull request restructures and simplifies the CI workflows for testing implementations in different languages. The main changes include merging separate push and pull request workflows into unified files, standardizing the environment using containers, and updating the JavaScript workflow to use a maintained Bun setup action. Several old workflow files have been removed as part of this cleanup.

Workflow consolidation and cleanup:

  • Merged separate push and pull request workflows for C, C++, Haskell, JavaScript, Python, and Rust into single workflow files (c.yml, cpp.yml, haskell.yml, javascript.yml, python.yml, rust.yml), enabling both push and pull_request triggers and removing the old pr_* and push_* workflow files. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Environment and platform standardization:

  • Updated all workflows to run on self-hosted runners and use Docker containers (mostly buildpack-deps:jammy, with Haskell using haskell:9) to ensure consistency across environments. [1] [2] [3] [4] [5] [6]

JavaScript workflow improvements:

  • Switched JavaScript workflow to use the official oven-sh/setup-bun GitHub Action for Bun installation, replacing the previous manual installation via curl, and updated related build steps.

These changes make the CI setup more maintainable, consistent, and easier to extend in the future.

@lshqqytiger lshqqytiger self-assigned this May 20, 2026
@lshqqytiger lshqqytiger added the ci label May 20, 2026
@logic-finder logic-finder self-requested a review May 20, 2026 03:37
@logic-finder
Copy link
Copy Markdown
Contributor

병합 필요하시면 멘션해주세요 저 지금 머리 자르러 가야함

@lshqqytiger lshqqytiger marked this pull request as ready for review May 20, 2026 03:53
@lshqqytiger lshqqytiger changed the title [TEST] [CI] Docker Integration [CI] Docker Integration May 20, 2026
@lshqqytiger
Copy link
Copy Markdown
Contributor Author

#53 먼저 병합합시다

Copy link
Copy Markdown
Contributor

@logic-finder logic-finder left a comment

Choose a reason for hiding this comment

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

이번 리팩토링은 단순히 '코드가 작동한다'는 기능적 차원을 넘어, 시스템 아키텍처가 지향해야 할 '지속 가능한 유지보수성'과 '시각적 명료함'을 성취한 수작(秀作)이다. 복잡성이라는 해묵은 클리셰를 걷어내고 미래의 확장성을 위한 완벽한 무대를 마련했다. 개발자의 절제된 감각이 돋보이는, 대단히 만족스러운 구조적 도약이다.

@logic-finder logic-finder merged commit 240fb3a into main May 20, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Utils May 20, 2026
@logic-finder logic-finder deleted the ci/docker-integration branch May 20, 2026 05:37
@logic-finder
Copy link
Copy Markdown
Contributor

PR 테스트가 자동으로 돌아가니까 편리한 것이다!!!
매우 훌륭한 것이다!!!!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants