Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

๐Ÿ”€ :: (#440) ๋„๋ฉ”์ธ ๋ชจ๋“ˆ ๋ถ„๋ฆฌ 8์ฐจ #469

Merged
merged 8 commits into from
Mar 11, 2024

Conversation

KangTaeHoon
Copy link
Contributor

@KangTaeHoon KangTaeHoon commented Mar 10, 2024

๐Ÿ’ก ๋ฐฐ๊ฒฝ ๋ฐ ๊ฐœ์š”

๋„๋ฉ”์ธ ๋ชจ๋“ˆ ๋ถ„๋ฆฌ 8์ฐจ (Last)
๋Œ€์ƒ: SongsDomain, PlayListDomain, UserDomain

Resolves: #440

๐Ÿ“ƒ ์ž‘์—…๋‚ด์šฉ

  • SongsDomain, PlayListDomain, UserDomain ๋ชจ๋“ˆ ์ถ”๊ฐ€ ๋ฐ ๋””ํŽœ๋˜์‹œ ์„ค์ •
  • Service ๋ชจ๋“ˆ์˜ ์ฝ”๋“œ SongsDomain, PlayListDomain, UserDomain๋กœ ์ด์ „
  • ๋ ˆ๊ฑฐ์‹œ ์ œ๊ฑฐ
  • tuist graph ์—…๋ฐ์ดํŠธ

๐Ÿ™‹โ€โ™‚๏ธ ๋ฆฌ๋ทฐ๋…ธํŠธ

SongEntity์™€ ๊ฐ™์€ ๊ณตํ†ต์œผ๋กœ ์‚ฌ์šฉํ•˜๋Š” entity๋•Œ๋ฌธ์— 3๊ฐ€์ง€ ๋„๋ฉ”์ธ ๋ถ„๋ฆฌ ์ž‘์—…์„ ํ•œ ๋ฒˆ์— ์ฒ˜๋ฆฌํ•˜์˜€์Šต๋‹ˆ๋‹ค.
์„œ๋น„์Šค ๋ ˆ์ด์–ด ์ œ๊ฑฐ๋Š” #442 ์—์„œ ์ง„ํ–‰ํ•ฉ๋‹ˆ๋‹ค.

image

โœ… PR ์ฒดํฌ๋ฆฌ์ŠคํŠธ

ํ…œํ”Œ๋ฆฟ ์ฒดํฌ๋ฆฌ์ŠคํŠธ ๋ง๊ณ ๋„ ์ถ”๊ฐ€์ ์œผ๋กœ ํ•„์š”ํ•œ ์ฒดํฌ๋ฆฌ์ŠคํŠธ๋Š” ์ถ”๊ฐ€ํ•ด์ฃผ์„ธ์š”!

  • ์ด ์ž‘์—…์œผ๋กœ ์ธํ•ด ๋ณ€๊ฒฝ์ด ํ•„์š”ํ•œ ๋ฌธ์„œ๊ฐ€ ๋ณ€๊ฒฝ๋˜์—ˆ๋‚˜์š”? (e.g. XCConfig, ๋…ธ์…˜, README)
  • ์ด ์ž‘์—…์„ ํ•˜๊ณ ๋‚˜์„œ ๊ณต์œ ํ•ด์•ผํ•  ํŒ€์›๋“ค์—๊ฒŒ ๊ณต์œ ๋˜์—ˆ๋‚˜์š”? (e.g. "API ๊ฐœ๋ฐœ ์™„๋ฃŒ๋์–ด์š”", "XCConfig ๊ฐ’ ์ถ”๊ฐ€๋˜์—ˆ์–ด์š”")
  • ์ž‘์—…ํ•œ ์ฝ”๋“œ๊ฐ€ ์ •์ƒ์ ์œผ๋กœ ๋™์ž‘ํ•˜๋‚˜์š”?
  • Merge ๋Œ€์ƒ ๋ธŒ๋žœ์น˜๊ฐ€ ์˜ฌ๋ฐ”๋ฅธ๊ฐ€์š”?
  • PR๊ณผ ๊ด€๋ จ ์—†๋Š” ์ž‘์—…์ด ์žˆ์ง€๋Š” ์•Š๋‚˜์š”?

๐ŸŽธ ๊ธฐํƒ€

@github-actions github-actions bot added โšก๏ธ Performance ์„ฑ๋Šฅ ๊ฐœ์„  2๏ธโƒฃ Priority: Medium ์šฐ์„ ์ˆœ์œ„ ์ค‘ โ™ป๏ธ Refactor ์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง labels Mar 10, 2024
Copy link

github-actions bot commented Mar 10, 2024

โœ… Successful finished SwiftLint

Copy link

โœ… Assign ์ž๋™ ์ง€์ •์„ ์„ฑ๊ณตํ–ˆ์–ด์š”!

@KangTaeHoon

Base automatically changed from feature/439-module-domain-7round to develop March 11, 2024 04:57
@KangTaeHoon KangTaeHoon merged commit b3df8ac into develop Mar 11, 2024
1 check passed
@KangTaeHoon KangTaeHoon deleted the feature/440-module-domain-8round branch March 11, 2024 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2๏ธโƒฃ Priority: Medium ์šฐ์„ ์ˆœ์œ„ ์ค‘ โšก๏ธ Performance ์„ฑ๋Šฅ ๊ฐœ์„  โ™ป๏ธ Refactor ์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง
Projects
None yet
Development

Successfully merging this pull request may close these issues.

๋„๋ฉ”์ธ ๋ชจ๋“ˆ ๋ถ„๋ฆฌ 8์ฐจ
3 participants