Skip to content

ci: add windows tests #385

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
medyagh wants to merge 5 commits into from
Closed

ci: add windows tests #385

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8fc8e24
ci: add windows tests
medyagh Jan 15, 2026
14c22a3
fix windows tests
medyagh Jan 15, 2026
ff4fd73
I have applied the fix by adding shell: bash to the workflow step. Th…
medyagh Jan 15, 2026
79adef0
created .gitattributes to enforce LF line endings. This should fix th…
medyagh Jan 15, 2026
73081a1
Merge branch 'master' into add_windows
medyagh Jan 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
strategy:
matrix:
go-version: [1.21.x]
os: [ubuntu-latest, macos-latest]
os: [ubuntu-latest, macos-latest, windows-latest]
runs-on: ${{ matrix.os }}
steps:
- name: Install Go
Expand All @@ -20,4 +20,5 @@ jobs:
run: go test -v -race ./...
- name: Format
if: matrix.go-version == '1.21.x'
shell: bash
run: diff -u <(echo -n) <(gofmt -d .)
3 changes: 2 additions & 1 deletion cmp/compare_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,8 @@ func mustParseGolden(path string) map[string]string {
if err != nil {
panic(err)
}
s := string(b)
// Normalize CRLF to LF to support Windows.
s := strings.ReplaceAll(string(b), "\r\n", "\n")

out := map[string]string{}
for len(s) > 0 {
Expand Down
Loading