Skip to content

fix: API 사용량 표 요청 번호에서 # 제거 (PR 자동 링크 회피)#41

Merged
soobing merged 1 commit intomainfrom
fix/usage-table-pr-autolink
Apr 27, 2026
Merged

fix: API 사용량 표 요청 번호에서 # 제거 (PR 자동 링크 회피)#41
soobing merged 1 commit intomainfrom
fix/usage-table-pr-autolink

Conversation

@soobing
Copy link
Copy Markdown
Contributor

@soobing soobing commented Apr 27, 2026

Summary

🔢 API 사용량 표의 요청 번호를 #1, #2, #3 으로 적었더니 GitHub 가 PR/issue 자동 링크로 인식해서 실제 PR #1, PR #2 로 연결되고 있었습니다. 헤더가 이미 요청 이라 # 없이 숫자만 적어도 의미가 분명하므로 그쪽으로 정리했습니다.

  • utils/learningComment.js: #${i + 1}${i + 1}
  • tests/learningComment.test.js: 표 매칭 expect 4개 갱신

Test plan

  • bun test tests/learningComment.test.js — 4 pass
  • 머지 후 실제 리트코드 PR 에서 🔢 API 사용량 표가 자동 링크 없이 일반 숫자로 표시되는지 확인

🤖 Generated with Claude Code

`#1`, `#2` 형태가 GitHub의 PR/issue 자동 링크 대상이 되어 실제 PR
#1, #2 로 잘못 연결되던 문제. 헤더가 이미 "요청"이라 의미가 분명하므로
숫자만 표기한다.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: sounmind <37020415+sounmind@users.noreply.github.com>
@soobing soobing requested a review from sounmind April 27, 2026 13:03
@soobing soobing self-assigned this Apr 27, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 7ef45a3 Commit Preview URL

Branch Preview URL
Apr 27 2026, 01:03 PM

@soobing soobing merged commit 8e421c9 into main Apr 27, 2026
2 checks passed
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.

2 participants