Conversation
|
The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).
|
| # G404 checks for use of the ordinary non-CPRNG. | ||
| path: private/buf/buflsp/progress.go | ||
| text: "G404:" | ||
| - linters: |
There was a problem hiding this comment.
@doriable can you take a look to see if these exceptions make sense?
There was a problem hiding this comment.
Yep -- so truncating conversions is definitely a known false positive in the issue I put for the G115 exemptions. I went through each one and they are all good. We already have a clear comment for the one in report.go also for converting the version, so this looks good to me.
| # G404 checks for use of the ordinary non-CPRNG. | ||
| path: private/buf/buflsp/progress.go | ||
| text: "G404:" | ||
| - linters: |
There was a problem hiding this comment.
Yep -- so truncating conversions is definitely a known false positive in the issue I put for the G115 exemptions. I went through each one and they are all good. We already have a clear comment for the one in report.go also for converting the version, so this looks good to me.
No description provided.