Skip to content

fix: SyntaxError: f-string: expecting '}' but got "}"#57

Merged
Soju06 merged 1 commit into
Soju06:mainfrom
tasoo-oos:main
Nov 14, 2024
Merged

fix: SyntaxError: f-string: expecting '}' but got "}"#57
Soju06 merged 1 commit into
Soju06:mainfrom
tasoo-oos:main

Conversation

@tasoo-oos
Copy link
Copy Markdown
Contributor

๐Ÿ› ๏ธ PR Summary

๐ŸŒŸ ์š”์•ฝ

์–ด๋–ค ๊ฒƒ์ด ๋ณ€๊ฒฝ๋˜์—ˆ๋‚˜์š”? ๊ฐ„๋žตํžˆ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.

๋‹ค์Œ ์˜ค๋ฅ˜๋ฅผ ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

  File "/home/tasoo1118/.miniconda3/envs/algotrading/lib/python3.11/site-packages/pykis/event/handler.py", line 129
    return f"{self.__class__.__name__}({", ".join(repr(filter) for filter in self.filters)}, gate={self.gate!r})"
                                         ^
SyntaxError: f-string: expecting '}'

๐Ÿ“Š ์ฃผ์š” ๋ณ€๊ฒฝ ์‚ฌํ•ญ

์ฃผ์š” ๋ณ€๊ฒฝ ์‚ฌํ•ญ์„ ์ ์–ด์ฃผ์„ธ์š”.

ํ•ด๋‹น ์ค„์˜ ๋‚ด๋ถ€ ์Œ๋”ฐ์˜ดํ‘œ๋ฅผ ์™ธ๋”ฐ์Œํ‘œ๋กœ ๋ณ€๊ฒฝํ–ˆ์Šต๋‹ˆ๋‹ค.

return f"{self.__class__.__name__}({", ".join(repr(filter) for filter in self.filters)}, gate={self.gate!r})"
return f"{self.__class__.__name__}({', '.join(repr(filter) for filter in self.filters)}, gate={self.gate!r})"

๐ŸŽฏ ๋ชฉ์  ๋ฐ ์˜ํ–ฅ

  • ๋ชฉ์ : ์™œ ์ด PR์ด ํ•„์š”ํ•œ๊ฐ€์š”?
    ํŒŒ์ด์ฌ 3.11 ๋ฒ„์ „์—์„œ ์—๋Ÿฌ๊ฐ€ ๋‚ฉ๋‹ˆ๋‹ค.

  • ์˜ํ–ฅ: ์ด ๋ณ€๊ฒฝ ์‚ฌํ•ญ์ด ์–ด๋–ค ์˜ํ–ฅ์„ ๋ฏธ์น˜๋‚˜์š”?
    ์ด ๋ณ€๊ฒฝ์‚ฌํ•ญ์ด ์ตœ์„ ์ธ์ง€๋Š” ์ž˜ ๋ชจ๋ฅด๊ฒ ์Šต๋‹ˆ๋‹ค.
    ๊ทธ๋ž˜๋„ ์—๋Ÿฌ๋Š” ์ด์ œ ์•ˆ ๋‚ฉ๋‹ˆ๋‹ค.

@Soju06 Soju06 added the ๋ฒ„๊ทธ ๋ญ”๊ฐ€ ์ž‘๋™ํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค. label Nov 14, 2024
@Soju06 Soju06 merged commit 7059918 into Soju06:main Nov 14, 2024
@Soju06
Copy link
Copy Markdown
Owner

Soju06 commented Nov 14, 2024

๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ์— ๊ธฐ์—ฌํ•ด์ฃผ์…”์„œ ๊ฐ์‚ฌํ•ฉ๋‹ˆ๋‹ค!

@Soju06 Soju06 self-requested a review November 14, 2024 00:35
Soju06 added a commit that referenced this pull request Feb 20, 2026
fix: SyntaxError: f-string: expecting '}' but got "}"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

๋ฒ„๊ทธ ๋ญ”๊ฐ€ ์ž‘๋™ํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants