Thanks to visit codestin.com
Credit goes to github.com

Skip to content

PR์—์„œ ํ”„๋กœ์ ํŠธ Week ์„ค์ • ๋ˆ„๋ฝ์„ ๋ฐฉ์ง€๋ฅผ ์œ„ํ•œ CI ์„ค์ •#1957

Closed
DaleSeo wants to merge 5 commits into
DaleStudy:mainfrom
DaleSeo:project-validation
Closed

PR์—์„œ ํ”„๋กœ์ ํŠธ Week ์„ค์ • ๋ˆ„๋ฝ์„ ๋ฐฉ์ง€๋ฅผ ์œ„ํ•œ CI ์„ค์ •#1957
DaleSeo wants to merge 5 commits into
DaleStudy:mainfrom
DaleSeo:project-validation

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Oct 26, 2025

Copy link
Copy Markdown
Member

๋‹ต์•ˆ ์ œ์ถœ ๋ฌธ์ œ

  • ๋ฌธ์ œ 1
  • ๋ฌธ์ œ 2
  • ๋ฌธ์ œ 3

์ž‘์„ฑ์ž ์ฒดํฌ ๋ฆฌ์ŠคํŠธ

  • Projects์˜ ์˜ค๋ฅธ์ชฝ ๋ฒ„ํŠผ(โ–ผ)์„ ๋ˆŒ๋Ÿฌ ํ™•์žฅํ•œ ๋’ค, Week๋ฅผ ํ˜„์žฌ ์ฃผ์ฐจ๋กœ ์„ค์ •ํ•ด์ฃผ์„ธ์š”.
  • ๋ฌธ์ œ๋ฅผ ๋ชจ๋‘ ํ‘ธ์‹œ๋ฉด ํ”„๋กœ์ ํŠธ์—์„œ Status๋ฅผ In Review๋กœ ์„ค์ •ํ•ด์ฃผ์„ธ์š”.
  • ์ฝ”๋“œ ๊ฒ€ํ† ์ž 1๋ถ„ ์ด์ƒ์œผ๋กœ๋ถ€ํ„ฐ ์Šน์ธ์„ ๋ฐ›์œผ์…จ๋‹ค๋ฉด PR์„ ๋ณ‘ํ•ฉํ•ด์ฃผ์„ธ์š”.

๊ฒ€ํ† ์ž ์ฒดํฌ ๋ฆฌ์ŠคํŠธ

Important

๋ณธ์ธ ๋‹ต์•ˆ ์ œ์ถœ ๋ฟ๋งŒ ์•„๋‹ˆ๋ผ ๋‹ค๋ฅธ ๋ถ„ PR ํ•˜๋‚˜ ์ด์ƒ์„ ๋ฐ˜๋“œ์‹œ ๊ฒ€ํ† ๋ฅผ ํ•ด์ฃผ์…”์•ผ ํ•ฉ๋‹ˆ๋‹ค!

  • ๋ฐ”๋กœ ์ด์ „์— ์˜ฌ๋ผ์˜จ PR์— ๋ณธ์ธ์„ ์ฝ”๋“œ ๋ฆฌ๋ทฐ์–ด๋กœ ์ถ”๊ฐ€ํ•ด์ฃผ์„ธ์š”.
  • ๋ณธ์ธ์ด ๊ฒ€ํ† ํ•ด์•ผํ•˜๋Š” PR์˜ ๋‹ต์•ˆ ์ฝ”๋“œ์— ํ”ผ๋“œ๋ฐฑ์„ ์ฃผ์„ธ์š”.
  • ํ† ์š”์ผ ์ „๊นŒ์ง€ PR์„ ๋ณ‘ํ•ฉํ•  ์ˆ˜ ์žˆ๋„๋ก ์Šน์ธํ•ด์ฃผ์„ธ์š”.

TonyKim9401
TonyKim9401 previously approved these changes Oct 27, 2025

@TonyKim9401 TonyKim9401 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

๋งค๋ฒˆ ํ™•์ธํ•œ๋‹ค๋Š”๊ฑธ ๊นŒ๋จน๋„ค์š”..์ฃ„์†กํ•ฉ๋‹ˆ๋‹ค..

@SamTheKorean

Copy link
Copy Markdown
Contributor

ci์„ค์ •์„ ํ•˜๋ฉด ๋งค๋ฒˆ ์ˆ˜๊ธฐ๋กœ ๋ณ€๊ฒฝํ•˜๋Š” ๋ฒˆ๊ฑฐ๋กœ์›€์ด ์‚ฌ๋ผ์งˆ ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค! ๊ธฐ์—ฌ๊ฐ์‚ฌํ•ฉ๋‹ˆ๋‹ค!

@DaleSeo

DaleSeo commented Nov 2, 2025

Copy link
Copy Markdown
Member Author

๋งค๋ฒˆ ํ™•์ธํ•œ๋‹ค๋Š”๊ฑธ ๊นŒ๋จน๋„ค์š”..์ฃ„์†กํ•ฉ๋‹ˆ๋‹ค..

@TonyKim9401 ๋‹˜, ์•„์ง Draft PR์ธ๋ฐ ์™œ ์Šน์ธํ•˜์‹œ๋Š” ๊ฑฐ์—์š” ใ…Žใ…Ž fork ์ €์žฅ์†Œ๋กœ ๋ถ€ํ„ฐ ๋งŒ๋“  PR์€ ํ”„๋กœ์ ํŠธ ๋ฐ์ดํ„ฐ์— ์ ‘๊ทผํ•  ์ˆ˜ ์—†๋Š” ๋ฌธ์ œ์— ๋ถ€๋”ชํ˜€์„œ ์•„์ง ํ•ด๊ฒฐํ•˜์ง€ ๋ชปํ–ˆ์Šต๋‹ˆ๋‹ค. ๋‹ค์Œ ๊ธฐ์ˆ˜ ์‹œ์ž‘ ์ „๊นŒ์ง€ ์™„๋ฃŒํ•˜๋Š” ๊ฒƒ์ด ๋ชฉํ‘œ์ž…๋‹ˆ๋‹ค.

@TonyKim9401 TonyKim9401 dismissed their stale review November 2, 2025 16:00

์ œ ์‹ค์ˆ˜์ž…๋‹ˆ๋‹ค ์ฒ ํšŒํ•ด๋‘๊ฒ ์Šต๋‹ˆ๋‹ค.

@DaleSeo

DaleSeo commented Nov 4, 2025

Copy link
Copy Markdown
Member Author

ํ˜„์žฌ ๋ฌธ์ œ ์š”์•ฝ

  • GitHub Actions์—์„œ ์‚ฌ์šฉํ•˜๋Š” GITHUB_TOKEN์„ ๊ฐ€์ง€๊ณ ๋Š” ํ”„๋กœ์ ํŠธ ๋ฐ์ดํ„ฐ์— ์ ‘๊ทผํ•  ์ˆ˜ ์—†์Œ
  • Secrects์— PAT๋‚˜ GitHub App ํ† ํฐ์„ ์ €์žฅํ•ด๋‘๊ณ  ๋ถˆ๋Ÿฌ์˜ค๋ฉด ํ”„๋กœ์ ํŠธ ๋ฐ์ดํ„ฐ์— ์ ‘๊ทผํ•  ์ˆ˜ ์žˆ์œผ๋‚˜, fork ์ €์žฅ์†Œ์˜ ๋ธŒ๋žœ์น˜๋ฅผ ๋ณ‘ํ•ฉํ•˜๊ธฐ ์œ„ํ•œ PR์—์„œ๋Š” ๋ณด์•ˆ์ƒ Secrects์— ์ ‘๊ทผํ•  ์ˆ˜ ์žˆ์Œ
  • pull_request_target ์ด๋ฒคํŠธ๋ฅผ ํ†ตํ•ด ์ด๋Ÿฌํ•œ ์ œํ•œ์„ ์šฐํšŒํ•  ์ˆ˜๋Š” ์žˆ์ง€๋งŒ ์œ„ํ—˜ํ•œ ์„ ํƒ

@DaleSeo

DaleSeo commented Nov 5, 2025

Copy link
Copy Markdown
Member Author

๋˜ ๋‹ค๋ฅธ ๋ฌธ์ œ๋“ค

  • GitHub Actions๋Š” Projects ๋ณ€๊ฒฝ์„ ์ง์ ‘ ๊ฐ์ง€ํ•  ์ˆ˜ ์—†์Œ

@DaleSeo DaleSeo force-pushed the project-validation branch from d9515d1 to 8813c5c Compare November 5, 2025 00:37
@DaleSeo DaleSeo force-pushed the project-validation branch from 8813c5c to 39f012f Compare November 5, 2025 00:44
@DaleSeo

DaleSeo commented Nov 5, 2025

Copy link
Copy Markdown
Member Author

/check-week

@DaleSeo DaleSeo force-pushed the project-validation branch from 223f32e to 045774d Compare November 5, 2025 00:52
PR comments use GitHub Issues API internally, so issues:write permission is required.
- Suppress stderr output for gh commands
- Continue workflow even if comment posting fails
- Add clearer error messages
@github-actions

github-actions Bot commented Nov 5, 2025

Copy link
Copy Markdown

โš ๏ธ Week ์„ค์ •์ด ๋ˆ„๋ฝ๋˜์—ˆ์Šต๋‹ˆ๋‹ค

ํ”„๋กœ์ ํŠธ์—์„œ Week๋ฅผ ์„ค์ •ํ•ด์ฃผ์„ธ์š”!

์„ค์ • ๋ฐฉ๋ฒ•

  1. PR ์šฐ์ธก์˜ Projects ์„น์…˜์—์„œ ๋ฆฌํŠธ์ฝ”๋“œ ์Šคํ„ฐ๋”” ์˜† ๋“œ๋กญ๋‹ค์šด(โ–ผ) ํด๋ฆญ
  2. ํ˜„์žฌ ์ฃผ์ฐจ๋ฅผ ์„ ํƒํ•ด์ฃผ์„ธ์š” (์˜ˆ: Week 14(current) ๋˜๋Š” Week 14)

๐Ÿ“š ์ž์„ธํ•œ ๊ฐ€์ด๋“œ ๋ณด๊ธฐ


๐Ÿค– ์ด ๋Œ“๊ธ€์€ GitHub App์„ ํ†ตํ•ด ์ž๋™์œผ๋กœ ์ž‘์„ฑ๋˜์—ˆ์Šต๋‹ˆ๋‹ค.

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