tangled.org / core
Monorepo for Tangled tangled.org
fork atom

nix: comment out cbor marshaler/unmarshaler before goimport check #1016

merged opened by boltless.me targeting master from sl/kkxwtpwumylk
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3mda6bcpp3h22
+3
Diff #0
unified split
+3
flake.nix
··· 284 rm -f api/tangled/* 285 lexgen --build-file lexicon-build-config.json lexicons 286 sed -i.bak 's/\tutil/\/\/\tutil/' api/tangled/* 287 ${pkgs.gotools}/bin/goimports -w api/tangled/* 288 go run ./cmd/cborgen/ 289 lexgen --build-file lexicon-build-config.json lexicons
··· 284 rm -f api/tangled/* 285 lexgen --build-file lexicon-build-config.json lexicons 286 sed -i.bak 's/\tutil/\/\/\tutil/' api/tangled/* 287 + # lexgen generates incomplete Marshaler/Unmarshaler for union types 288 + find api/tangled/*.go -not -name "cbor_gen.go" -exec \ 289 + sed -i '/^func.*\(MarshalCBOR\|UnmarshalCBOR\)/,/^}/ s/^/\/\/ /' {} + 290 ${pkgs.gotools}/bin/goimports -w api/tangled/* 291 go run ./cmd/cborgen/ 292 lexgen --build-file lexicon-build-config.json lexicons

History

1 round 2 comments
sign up or login to add to the discussion
boltless.me submitted #0
1 commit
expand
72dc2a5e
nix: comment out cbor marshaler/unmarshaler before goimport check
3/3 success
expand
build.yml
fmt.yml
test.yml
expand 2 comments
oppi.li

strange that it does this! might be worth opening an issue for on upstream.

boltless.me

I guess it is just expected behavior in current version. I'm just waiting for ongoing lexgen rewrite to be finished.

pull request successfully merged