Skip to content

Check out shell scripts with Unix line endings on Windows#833

Merged
tjementum merged 1 commit intoplatformplatform:mainfrom
kimsey0:ClaudeHooksLineEndings
Mar 8, 2026
Merged

Check out shell scripts with Unix line endings on Windows#833
tjementum merged 1 commit intoplatformplatform:mainfrom
kimsey0:ClaudeHooksLineEndings

Conversation

@kimsey0
Copy link
Contributor

@kimsey0 kimsey0 commented Jan 19, 2026

Summary & Motivation

Updates .gitattributes to check out .sh files with Unix line endings (\n) on all platforms, including Windows.

Checklist

  • I have added tests, or done manual regression tests
  • I have updated the documentation, if necessary

@kimsey0 kimsey0 requested a review from a team as a code owner January 19, 2026 10:20
@CLAassistant
Copy link

CLAassistant commented Jan 19, 2026

CLA assistant check
All committers have signed the CLA.

@tjementum
Copy link
Member

I will think about this and see if I can find a better solution. I have been successfully making the .gitattributes only have '* text=auto'. It might be that on Windows, bash scripts are checked out with 'CRLF', but since Claude Code is running in WSL, it does not work. So maybe there is no way to avoid this. Ideally, I would love to delete the .gitattributes alltogehter.

@tjementum tjementum self-assigned this Jan 19, 2026
@tjementum tjementum added the Bug Something isn't working label Jan 19, 2026
@tjementum tjementum force-pushed the ClaudeHooksLineEndings branch from 96edd67 to 6c5951b Compare March 8, 2026 11:21
@tjementum tjementum enabled auto-merge March 8, 2026 11:21
@tjementum tjementum disabled auto-merge March 8, 2026 11:21
@tjementum tjementum merged commit 9e822f7 into platformplatform:main Mar 8, 2026
1 check failed
@github-project-automation github-project-automation bot moved this to ✅ Done in Kanban board Mar 8, 2026
@tjementum tjementum linked an issue Mar 8, 2026 that may be closed by this pull request
2 tasks
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

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Claude hooks broken by Windows line endings

3 participants