Skip to content

Commit

Permalink
Update ContributingOnWindows.md (#744)
Browse files Browse the repository at this point in the history
  • Loading branch information
ambiguousname authored Dec 8, 2024
1 parent 42f8e40 commit effebbb
Showing 1 changed file with 17 additions and 1 deletion.
18 changes: 17 additions & 1 deletion ContributingOnWindows.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,20 @@ For full windows support, it'd be nice to have `npm` as a variable that switches
npm = which npm
```

You also don't have to use Cargo make. VS Code tasks also help me circumvent this kind of stuff. But I try to avoid calling npm directly if possible, since everybody should ideally be using the same build tools.
You also don't have to use Cargo make. VS Code tasks also help me circumvent this kind of stuff. But I try to avoid calling npm directly if possible, since everybody should ideally be using the same build tools.

## CRLF vs. LF
### Random Whitespacing in Generation (core.autocrlf)
Git, in its attempts to be helpful, created a bunch of extra whitespaces in converting CRLF to LF. Which led to differences from the CI on Windows.

So these commands help get rid of that:

```bash
# Disable Git's CRLF to LF conversion:
git config --global core.autocrlf false
# Reset cached files to remove associations
git rm --cached -r .
git reset --hard
```

You can re-enable the `core.autocrlf` option afterwards if you'd like: `git config --global core.autocrlf true`. Just don't reset the cache.

0 comments on commit effebbb

Please sign in to comment.