jf-tech / omniparser

omniparser: a native Golang ETL streaming parser and transform library for CSV, JSON, XML, EDI, text, etc.
MIT License
931 stars 68 forks source link

Error in go get install latest version #176

Closed deokapil closed 2 years ago

deokapil commented 2 years ago

Hi,

I am getting following error while installing the latest version v1.0.2 of omniparser

extensions/omniv21/fileformat/flatfile/fixedlength/.snapshots/TestReadAndMatchRowsBasedEnvelope-non-empty_buf;_no_read;_match;_create_IDR: malformed file path "extensions/omniv21/fileformat/flatfile/fixedlength/.snapshots/TestReadAndMatchRowsBasedEnvelope-non-empty_buf;_no_read;_match;_create_IDR": invalid char ';'

probably accidental typo in file name is checked in

jf-tech commented 2 years ago

Are you running on Windows?

jf-tech commented 2 years ago

Okay, not related to your OS. It's from here:

https://cs.opensource.google/go/x/mod/+/master:module/module.go;l=285;drc=02c991387e35f9e4c0a9b7ce137717055f8d716d

Basically module file names have a reduced set of valid characters. And the two recently introduced snapshot filenames contain ; and while it's all good in coding, testing, and CI, it's failing during import.

I'll scrub all the use of ; from all test names.

jf-tech commented 2 years ago

Also filed a ticket with https://github.com/bradleyjkemp/cupaloy/issues/83

jf-tech commented 2 years ago

The issue should be fixed now with release of 1.0.3. Give it another try.