Skip to content

fix(IS-5285): use accepted spellings in Buzzvil README - #28

Merged
luis-hong merged 1 commit into
version-2026.8from
IS-5285/fix-spellcheck
Sep 28, 2026
Merged

luis-hong merged 1 commit into
version-2026.8from
IS-5285/fix-spellcheck

Conversation

@luis-hong

Copy link
Copy Markdown

Summary

Test plan

  • npx --yes cspell@10.0.1 .buzzvil/README.md --config cspell.config.jsonc --no-progress
  • git diff --check

Linear Issue

https://linear.app/buzzvil/issue/IS-5285

@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 9aa554e5-26cb-492b-902c-03c0d223ae2c

📥 Commits

Reviewing files that changed from the base of the PR and between 5ec5973 and 3599f52.

📒 Files selected for processing (1)
  • .buzzvil/README.md

Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

README의 OAuth2 CORS 설명에서 honour를 honor로, behaviour를 behavior로 바꿨습니다. CORS Origin 검사, 정규식 처리 및 OPTIONS 동작에 관한 설명은 변경하지 않았습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 3599f

The README wording changes do not affect behavior, so no actionable merge risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 .buzzvil/README.md의 철자 수정이라는 주요 변경 사항을 명확하게 설명합니다. 이슈 식별자와 변경 목적도 포함합니다.
Description check ✅ Passed 설명은 변경 내용, 변경 이유, 테스트 결과, 연결된 이슈를 포함합니다. 템플릿의 Details 및 Checklist 제목과 체크리스트는 없지만, 핵심 정보는 대부분 제공됩니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@luis-hong
luis-hong merged commit 456a4b9 into version-2026.8 Sep 28, 2026
80 of 93 checks passed
@luis-hong
luis-hong deleted the IS-5285/fix-spellcheck branch September 28, 2026 09:28
@hugolee-woobuntu

Copy link
Copy Markdown

요약

🔴 하나에 답이 필요합니다. 이 PR 은 .buzzvil/README.md 의 영국식 철자 둘을 고치지만, CI 의 spellcheck 가 거부하는 자리는 그 파일이 아니라 authentik/providers/oauth2/tests/test_utils.py 라서 이 PR head 에서도 base 와 똑같은 2건으로 여전히 빨갛습니다.

확인 부탁드리는 것

  • 🔴 authentik/providers/oauth2/tests/test_utils.py:97,134 — CI 가 거부하는 것은 여기 behaviour 둘이고, README 는 cspell . 의 스캔 대상이 아닙니다. 이 두 줄을 behavior 로 바꾸는 것이 본문의 목적에 맞아 보이는데, README 만 고친 것이 의도인지 확인 부탁드립니다.
  • 🟡 ldap · rac · radius · proxy 컨테이너 빌드 8개 — base 커밋 5ec5973 에서도 같은 자리(ghcr.io 캐시 push 권한 거부)에서 실패하고 있어 이 PR 과 무관합니다. version-2026.8 에는 브랜치 보호와 ruleset 이 없어 머지를 막지도 않습니다.
근거 — 무엇을 돌려서 알았나

🔴 test_utils.py:97,134

PR head 3599f52 의 lint (spellcheck, node) 잡 로그(job 108866449094):

> cspell . --config cspell.config.jsonc
authentik/providers/oauth2/tests/test_utils.py:97:64 - Forbidden word (behaviour) fix: (behavior)
authentik/providers/oauth2/tests/test_utils.py:134:31 - Forbidden word (behaviour) fix: (behavior)
CSpell: Files checked: 4541, skipped: 5, Issues found: 2 in 1 file.

base 5ec5973 의 같은 잡(job 108864798194)도 Issues found: 2 in 1 file 로 동일합니다. 스캔된 4,541개 경로 중 . 으로 시작하는 것이 0개라(.github/ 도 없음) .buzzvil/README.md 는 CI 가 보지 않습니다. 테스트 플랜의 npx cspell .buzzvil/README.md 는 파일을 직접 지정하므로 통과하고, 그 차이가 로컬 초록과 CI 빨강을 갈랐습니다.

cspell.config.jsonc 의 flagWords 에 behaviour->behavior 가 있고, python 은 includeRegExpList: ["comments"] 라 주석·docstring 만 검사합니다. 같은 파일의 honoured(11행)·honours(150행)는 flagWords 에 없어 걸리지 않습니다. test_utils.py 는 #27 에서 새로 추가된 파일입니다.

🟡 컨테이너 빌드

#24 ERROR: error writing layer blob: denied: permission_denied: The requested installation does not exist.

base 커밋 check-runs 에서 build (ldap|rac|radius|proxy) / build-(amd64|arm64) 8개가 전부 failure 입니다. gh api repos/Buzzvil/authentik/branches/version-2026.8/protection 은 404 Branch not protected, rules/branches/version-2026.8 은 0개입니다.

알 수 없는 것

없습니다.

어디까지 봤나

diff · 본문 · PR #27 의 변경 파일 목록 · PR head 와 base 커밋의 check-runs · spellcheck 잡 로그 둘 · arm64 ldap 빌드 로그 · cspell.config.jsonc 전문 · .buzzvil/README.md 와 test_utils.py 의 head 실물 · 브랜치 보호와 ruleset. cspell 을 로컬에서 돌리지는 않았고 CI 로그를 근거로 썼습니다.


Hugo 요청으로 Claude 가 조사했습니다. 승인 판단은 Hugo 가 합니다.

@codecov

codecov Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.29%. Comparing base (5ec5973) to head (3599f52).
⚠️ Report is 1 commits behind head on version-2026.8.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                Coverage Diff                 @@
##           version-2026.8      #28      +/-   ##
==================================================
- Coverage           91.29%   91.29%   -0.01%     
==================================================
  Files                1162     1162              
  Lines               74961    74961              
  Branches             4056     4056              
==================================================
- Hits                68438    68433       -5     
- Misses               6481     6486       +5     
  Partials               42       42              
Flag Coverage Δ
conformance 33.75% <ø> (ø)
integration 30.41% <ø> (+<0.01%) ⬆️
rust 42.89% <ø> (ø)
unit 93.44% <ø> (ø)
unit-migrate 93.45% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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