Skip to content

fix : 오타로 인한 컴파일 오류 해결#19

Merged
Juhye0k merged 1 commit intodevfrom
wifi
Nov 15, 2025
Merged

fix : 오타로 인한 컴파일 오류 해결#19
Juhye0k merged 1 commit intodevfrom
wifi

Conversation

@Juhye0k
Copy link
Copy Markdown
Contributor

@Juhye0k Juhye0k commented Nov 15, 2025

🚀 1. 개요

컴파일 에러 수정

Summary by CodeRabbit

Bug Fixes

  • WiFi 네트워크 검증 기능의 스코핑 문제를 수정하여 올바른 작동을 보장합니다.

Chores

  • 사용되지 않는 import를 제거하여 코드를 정리했습니다.

@Juhye0k Juhye0k requested a review from kon28289 November 15, 2025 05:37
@Juhye0k Juhye0k self-assigned this Nov 15, 2025
@Juhye0k Juhye0k added the bug Something isn't working label Nov 15, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Nov 15, 2025

개요

CampusWiFiProperties 파일에서 미사용 import(ConstructorBinding)를 제거하고 WiFiNetwork 레코드 내부의 불필요한 닫는 괄호를 삭제했습니다. 검증 메서드의 범위와 논리는 유지됩니다.

변경사항

응집도 / 파일 변경 요약
코드 정리
src/main/java/com/gpt/geumpumtabackend/wifi/config/CampusWiFiProperties.java
미사용 ConstructorBinding import 제거 및 WiFiNetwork 레코드의 구조 개선 (불필요한 닫는 괄호 삭제)

검토 난이도 예측

🎯 1 (Trivial) | ⏱️ ~3분

  • 미사용 import 제거 및 구조적 정리만 포함
  • 비즈니스 로직 변경 없음
  • 단일 파일 변경

🐰 한 줄의 괄호가 사라지고,
쓸모 없는 import도 떠나가니,
WiFiNetwork는 더 깔끔하게 빛나네!
작은 정리가 모여,
코드의 정원을 가꾼다네. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ❓ Inconclusive 풀 리퀘스트 설명이 템플릿의 필수 섹션 중 일부만 포함되어 있습니다. '개요' 섹션만 작성되었고, '주요 변경 사항'과 '스크린샷' 섹션은 누락되었습니다. 설명을 보완하여 '📝 2. 주요 변경 사항' 섹션에 구체적인 변경 내용(import 제거, 중괄호 수정 등)을 추가하시기 바랍니다.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed 풀 리퀘스트 제목은 주요 변경 사항을 명확하게 설명합니다. 오타로 인한 컴파일 오류 해결이라는 내용이 코드 변경 사항(불필요한 import 제거 및 중괄호 오류 수정)과 일치합니다.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch wifi

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4743fa0 and 761cf76.

📒 Files selected for processing (1)
  • src/main/java/com/gpt/geumpumtabackend/wifi/config/CampusWiFiProperties.java (1 hunks)
🔇 Additional comments (2)
src/main/java/com/gpt/geumpumtabackend/wifi/config/CampusWiFiProperties.java (2)

6-6: 사용하지 않는 import 제거가 올바르게 처리되었습니다.

ConstructorBinding import가 제거되어 코드가 깔끔해졌습니다.


27-57: WiFiNetwork record의 구조가 올바르게 수정되었습니다.

추가 중괄호가 제거되어 모든 검증 메서드(isValidGatewayIP, isValidBSSID, isValidIP, isIpInRange)가 WiFiNetwork record 내에 올바르게 위치하게 되었습니다. 로직 변경 없이 컴파일 오류만 깔끔하게 해결되었습니다.


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.

Copy link
Copy Markdown
Contributor

@kon28289 kon28289 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

@Juhye0k Juhye0k merged commit c7821a7 into dev Nov 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants