Skip to content

Conversation

@gdaegeun539
Copy link
Member

@gdaegeun539 gdaegeun539 commented Nov 26, 2025

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines

What kind of change does this PR introduce?

Select your one and delete others

  • Bug fix

What is the current behavior?

You can also link to an open issue here

Windows에서 작업할 때 gradlew의 권한이 'rw-r--r--' 로 자꾸 바뀝니다.
맥에서 쉘로 detekt 확인할 때 매번 권한 바꿔줘야 해요...

What is the new behavior (if this is a feature change)?

Windows에서 작업할 때 gradlew와 추후 생성하는 .sh 확장자의 쉘 스크립트의 권한을 수정하지 않도록 합니다.
macOS(와 Linux)에서 작업할 때도 추후 생성하는 .bat 확장자의 쉘 스크립트의 줄바꿈(CRLF)을 유지하도록 합니다.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

No breaking changes in Windows.
macOS와 Linux에서는 더 이상 detekt의 권한을 다시 변경할 필요가 없습니다.

ScreenShots

스크린샷 2025-11-26 오후 4 45 05
(작업 할 때 마다 권한 바뀌니까 귀찮더라고요.)

Other information:

브랜칭 전략이 git flow랑 안 맞는 거 같은데, 이거는 기능도 아니고 핫픽스도 아니어서 일단 fix로 했어요. 다른 브랜치 이름이 적합할 지 아니면 이 카테고리를 추가할 지 한번 검토 부탁드려요.

- Change file mode from 644 to 755
- gradlew 실행 권한 문제 해결
- Preserve LF line endings for gradlew and *.sh files
- Prevent Windows from modifying file permissions
- Windows에서 작업해도 쉘 스크립트 권한이 유지됨
@gdaegeun539 gdaegeun539 self-assigned this Nov 26, 2025
@gdaegeun539 gdaegeun539 added build 빌드 관련 파일 수정 ci CI 관련 설정 수정 labels Nov 26, 2025
Copy link
Collaborator

@hyunjung-choi hyunjung-choi left a comment

Choose a reason for hiding this comment

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

fix 괜찮은 것 같아요 ! 수고하셨습니다 !

@gdaegeun539
Copy link
Member Author

fix 괜찮은 것 같아요 ! 수고하셨습니다 !

확인 감사합니다!

@gdaegeun539 gdaegeun539 merged commit cd3fef0 into project-lyrics:develop Nov 27, 2025
2 checks passed
@gdaegeun539 gdaegeun539 deleted the fix/gradlew-permisson-fix branch December 3, 2025 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build 빌드 관련 파일 수정 ci CI 관련 설정 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants