Skip to content

chore: rename app name#4

Merged
ik-523 merged 1 commit into
mainfrom
yunjunik06/cc-185-rename-app-name
Sep 15, 2025
Merged

chore: rename app name#4
ik-523 merged 1 commit into
mainfrom
yunjunik06/cc-185-rename-app-name

Conversation

@ik-523
Copy link
Copy Markdown
Collaborator

@ik-523 ik-523 commented Sep 15, 2025

PR 및 파일 구조 가이드라인

PR 가이드라인 (필독!) - Notion의 PR Guideline 참고
  • 커밋 가이드라인에 맞춰서 커밋을 해주시되, 한 PR에서 할당된 내용이 여러개이거나, 과정이 있을 경우 커밋을 여러 개로 나누어 작성 하시길 권장드립니다.
  • 만약 컴포넌트를 새로 만들거나, 디자인 관련 PR일 경우 스크린샷 혹은 작동 화면녹화를 PR 설명에 적어주세요.
  • Coderabbit (AI)이 PR을 생성할 경우 리뷰합니다. 봇 코멘트는 참고만 하시고, Resolve Conversation 누르시면 됩니다.
  • 각 프로젝트의 Lead 및 Sub-Lead (FE의 경우 @hoony6134, @rhseung) 검토자로 넣어주세요.
  • 2개의 approve가 있어야 main으로 머지가 가능합니다. 만약 Lead 혹은 Sub-Lead의 승인을 받았다면 @coderabbitai approve를 댓글에 작성하면 토끼가 승인을 눌러줘서 2개가 됩니다.
  • Flutter 프로젝트의 경우, 형식 및 위젯 테스트와 웹 버전 빌드 및 배포가 자동으로 진행됩니다. 테스트 및 웹 빌드/배포 절차를 거쳐 status check가 완료되어야 approve를 받고, main으로 머지될 수 있습니다. flutter test, dart analyze, flutter build web --release 명령을 실행하여 모두 정상적으로 작동되는지 확인해 보시길 권장드립니다.
파일 구조 가이드라인 (필독!) - Notion의 File Structure Guideline 참고
  • lib/main.dart 파일은 수정하지 말아 주세요.
  • 테스트 파일은 프로덕션 파일에 포함해서는 안됩니다.
  • 모든 파일 및 폴더는 snake_case 명명법을 따릅니다.
    • 옳은 예) auth/register/step_4.dart, main.dart
    • 나쁜 예) Views/component.dart, step-4.dart
  • 기능에 알맞는 폴더(view, service, model 등)에 파일을 추가해 주세요.

체크리스트

아래 체크리스트 내용을 확인하시고, 모든 항목에 체크해 주세요.

  • 파일 구조가 알맞게 작성되었습니다.
  • 커밋 가이드라인에 맞는 커밋 메시지를 작성하였습니다.
  • 작동 화면을 포함하지 않는 PR이거나, 아래 스크린샷/화면녹화를 업로드했습니다.

Summary by CodeRabbit

  • 스타일
    • 앱 전역 제목을 ‘CodiCue’로 변경하여 상단 바, 작업 전환기, 브라우저 탭 등에서 새 이름이 표시됩니다.
    • 홈 화면의 표시 제목을 ‘CodiCue Demo Home Page’로 업데이트해 명칭을 일관되게 반영합니다.
    • 기능, 동작, 설정 및 API에는 변화가 없으며 시각적 텍스트만 조정되었습니다.

@ik-523 ik-523 requested a review from a team as a code owner September 15, 2025 15:09
@linear
Copy link
Copy Markdown

linear Bot commented Sep 15, 2025

CC-185 Rename app name

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
codicue-app Ready Ready Preview Comment Sep 15, 2025 3:10pm

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Sep 15, 2025

Walkthrough

  • lib/main.dart에서 UI용 문자열만 변경됨.
  • MaterialApp의 title가 'Flutter Demo'에서 'CodiCue'로 수정됨.
  • MyHomePage 생성자에 전달되는 title이 'Flutter Demo Home Page'에서 'CodiCue Demo Home Page'로 변경됨.
  • 제어 흐름, 상태 관리, 공개 API 변경 없음.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • 단일 파일에서의 동질적인 문자열 치환만 포함됨.
  • 로직 변경, 구조 수정, 다중 패턴 없음.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed 제목 "chore: rename app name"은 PR의 핵심 변경 사항인 앱 이름 변경(예: lib/main.dart의 MaterialApp title과 MyHomePage 타이틀 문자열 수정)을 정확히 요약하고 있으며, 간결하고 명확한 커밋 컨벤션('chore:')을 따릅니다. 제목은 한 문장으로 충분히 구체적이고 오해의 소지가 없으며 불필요한 정보나 이모지를 포함하지 않습니다. 따라서 변경 내용과 일치하여 히스토리를 훑는 동료가 주요 변경점을 빠르게 이해할 수 있습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch yunjunik06/cc-185-rename-app-name

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@ik-523 ik-523 merged commit 520575a into main Sep 15, 2025
3 of 4 checks passed
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
lib/main.dart (1)

33-33: Home 페이지 제목의 ‘Demo’ 제거 권장(프로덕션 톤 정리)

사용자 노출 문자열에 ‘Demo’가 포함되어 있어 프로덕션 빌드에 부적절할 수 있습니다. 간단히 이름만 노출하거나 i18n으로 분리하세요.

-      home: const MyHomePage(title: 'CodiCue Demo Home Page'),
+      home: const MyHomePage(title: 'CodiCue'),

향후 i18n 도입 시:

  • onGenerateTitle과 동일하게 AppLocalizations를 사용해 제목을 제공하는 패턴을 권장합니다.
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 908feb6 and 295160c.

⛔ Files ignored due to path filters (1)
  • pubspec.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • lib/main.dart (2 hunks)
🔇 Additional comments (1)
lib/main.dart (1)

14-14: 앱 표시명 변경 — 플랫폼 리소스 동기화 및 가이드 준수 확인 필요

  • Notion 가이드에 'lib/main.dart 수정 금지'가 있다면 이번 변경이 예외로 승인됐는지 확인.
  • 검증 결과:
    • lib/main.dart — MaterialApp.title: 'CodiCue' (line 14); home: MyHomePage(title: 'CodiCue Demo Home Page') (line 33) — 'Demo' 잔여.
    • web/index.html — <title> = 'codicue_app'.
    • web/manifest.json — "name"/"short_name" = "codicue_app".
    • pubspec.yaml — name: codicue_app.
    • ios/Runner/Info.plist — CFBundleDisplayName 키 존재(값 확인 필요).
    • android/**/strings.xml — 검색 결과 없음(안드로이드 표시명 미동기화 가능성).
  • 조치(필수): 모든 플랫폼의 사용자 노출명(앱명/타이틀)을 일관되게 동기화하세요 — Android(strings.xml: app_name), iOS(Info.plist: CFBundleDisplayName), 웹(index.html <title>, manifest.json name/short_name), pubspec.yaml, 앱 내부 타이틀(MyHomePage)에서 'Demo' 제거.
  • 권장(국제화): MaterialApp.title 대신 onGenerateTitle 또는 l10n(AppLocalizations) 사용.
  • 선택적 리팩터(동작 동일):
-      title: 'CodiCue',
+      onGenerateTitle: (_) => 'CodiCue',

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