Skip to content

Conversation

@wlaud2000
Copy link
Member

๐Ÿ“ PR ํƒ€์ž… (ํ•˜๋‚˜ ์ด์ƒ ์„ ํƒ)

  • ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  • ๋ฒ„๊ทธ ์ˆ˜์ •
  • ์˜์กด์„ฑ, ํ™˜๊ฒฝ ๋ณ€์ˆ˜, ๋นŒ๋“œ ๊ด€๋ จ ์ฝ”๋“œ ์—…๋ฐ์ดํŠธ
  • ๊ธฐํƒ€ ์‚ฌ์†Œํ•œ ์ˆ˜์ •

โ—๏ธ ๊ด€๋ จ ์ด์Šˆ ๋งํฌ

Close #89

๐Ÿ“Œ ๊ฐœ์š”

  • IllegalArgumentException ์œผ๋กœ ๋˜์–ด์žˆ๋˜ ์˜ˆ์™ธ์ฒ˜๋ฆฌ๋ฅผ WeatherException์œผ๋กœ ๋ณ€๊ฒฝํ•˜์˜€์Šต๋‹ˆ๋‹ค.

๐Ÿ” ๋ณ€๊ฒฝ ์‚ฌํ•ญ

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท (Optional)

๐Ÿ‘€ ๊ธฐํƒ€ ๋” ์ด์•ผ๊ธฐํ•ด๋ณผ ์  (Optional)

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

  • PR ํ…œํ”Œ๋ฆฟ์— ๋งž์ถ”์–ด ์ž‘์„ฑํ–ˆ์–ด์š”.
  • ๋ณ€๊ฒฝ ๋‚ด์šฉ์— ๋Œ€ํ•œ ํ…Œ์ŠคํŠธ๋ฅผ ์ง„ํ–‰ํ–ˆ์–ด์š”.
  • ํ”„๋กœ๊ทธ๋žจ์ด ์ •์ƒ์ ์œผ๋กœ ๋™์ž‘ํ•ด์š”.
  • PR์— ์ ์ ˆํ•œ ๋ผ๋ฒจ์„ ์„ ํƒํ–ˆ์–ด์š”.
  • ๋ถˆํ•„์š”ํ•œ ์ฝ”๋“œ๋Š” ์‚ญ์ œํ–ˆ์–ด์š”.

@wlaud2000 wlaud2000 merged commit 6983912 into develop Aug 12, 2025
1 check passed
@wlaud2000 wlaud2000 deleted the fix/#89-weather-exception-fix branch August 12, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

๐Ÿ› [Fix] ๋‚ ์”จ API ์ด์Šˆ ์ฒ˜๋ฆฌ ์ˆ˜์ •

3 participants