uber-go / cadence-client

Framework for authoring workflows and activities running on top of the Cadence orchestration engine.
https://cadenceworkflow.io
MIT License
339 stars 128 forks source link

ran "make fmt" #1206

Closed mindaugasbarcauskas closed 1 year ago

mindaugasbarcauskas commented 1 year ago

What changed?

Just ran "make fmt" so that files are formatted correctly.

Why?

These changes come in when running "make build" and interfere when making changes.

How did you test it? Should be no functional impact.

Potential risks Not applicable.

Groxx commented 1 year ago

We should probably:

I'm assuming you did this with 1.19, but double check? >=1.19 and <1.19 produce different output, which is the underlying issue you're hitting.

mindaugasbarcauskas commented 1 year ago

I did it with 1.19 go version. Yeah I was wondering if gofmt might depend on local configuration. I will check on those points. Thanks Steven.

Groxx commented 1 year ago

I did it with 1.19 go version. Yeah I was wondering if gofmt might depend on local configuration. I will check on those points. Thanks Steven.

Normally it doesn't, but 1.19 made (good!) enhancements to godoc's capabilities and formatting... and unfortunately it thrashes with older versions :| Historically they've been fairly careful to make changes that are backwards compatible, but not this time.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 0185cb28-c2f3-49da-893b-efb14a042da2


Files with Coverage Reduction New Missed Lines %
internal/compatibility/thrift/enum.go 4 21.2%
internal/compatibility/thrift/types.go 9 48.42%
<!-- Total: 13 -->
Totals Coverage Status
Change from base Build 0185c943-fd48-4b30-94dd-4bd415ff8817: -0.06%
Covered Lines: 12684
Relevant Lines: 19744

💛 - Coveralls