Skip to content

하위 디렉터리 만들어서 Rust 코드 집어넣기#63

Merged
reddevilmidzy merged 5 commits intomainfrom
rook
May 16, 2025
Merged

하위 디렉터리 만들어서 Rust 코드 집어넣기#63
reddevilmidzy merged 5 commits intomainfrom
rook

Conversation

@reddevilmidzy
Copy link
Owner

♟️ What’s this PR about?

레포 하나에 작업할 수 있는 환경을 구성하였습니다.

🔗 Related Issues / PRs

part of #62

Copilot AI review requested due to automatic review settings May 16, 2025 16:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR creates a new subdirectory ("rook") for placing Rust code and updates CI/CD configurations to support it.

  • Introduces a .gitignore file in the rook directory to ignore build and IDE files.
  • Modifies the shuttle deployment workflow to include the new rook branch and sets its working directory.
  • Updates the Rust workflow to adjust caching, formatting, linting, building, and testing steps for the rook directory.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

File Description
rook/.gitignore Added ignore rules for build artifacts, IDE configs, and secret toml files
.github/workflows/shuttle_deploy.yml Updated branch triggers and deployment steps to support the new rook directory
.github/workflows/rust.yml Adjusted workflow steps (cache, cargo fmt, clippy, build, test) to use the rook directory

reddevilmidzy and others added 2 commits May 17, 2025 01:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@reddevilmidzy
Copy link
Owner Author

main <- rook 방향일 때 build가 두번 실행된다.
main 쪽으로 PR이 올라왔기 때문에, 그리고 rook에 push가 되었기 때문에.

main <- rook 방향일 때 build가 한번만 실행되도록 수정하자.

@reddevilmidzy reddevilmidzy merged commit 4ba0a0d into main May 16, 2025
5 checks passed
@reddevilmidzy reddevilmidzy added this to the 2차 MVP 구현 milestone Jun 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants