Skip to content

Commit

Permalink
This closes #1886, remove the namespace prefix for the default spread…
Browse files Browse the repository at this point in the history
…sheet namespace

- Improvement compatibility for the workbook internal part with a spreadsheet namespace prefix
- Update GitHub Action configuration, using the macOS 13 in the unit test pipeline to temporarily resolve test failed in macos-14-arm64
  • Loading branch information
xuri committed Apr 27, 2024
1 parent 055349d commit 7715c14
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ jobs:
strategy:
matrix:
go-version: [1.18.x, 1.19.x, 1.20.x, '>=1.21.1', 1.22.x]
os: [ubuntu-latest, macos-latest, windows-latest]
os: [ubuntu-latest, macos-13, windows-latest]
targetplatform: [x86, x64]

runs-on: ${{ matrix.os }}
Expand Down
5 changes: 5 additions & 0 deletions lib.go
Original file line number Diff line number Diff line change
Expand Up @@ -646,6 +646,11 @@ func getRootElement(d *xml.Decoder) []xml.Attr {
case xml.StartElement:
tokenIdx++
if tokenIdx == 1 {
for i := 0; i < len(startElement.Attr); i++ {
if startElement.Attr[i].Value == NameSpaceSpreadSheet.Value {
startElement.Attr[i] = NameSpaceSpreadSheet
}
}
return startElement.Attr
}
}
Expand Down

0 comments on commit 7715c14

Please sign in to comment.