search

containers / storage

Container Storage Library
Apache License 2.0
537 stars 234 forks source link

chunked: Fix two minor linter issues #1959

Closed cgwalters closed 3 weeks ago

cgwalters commented 3 weeks ago

My IDE runs a linter by default, and these two show up. For the file one, it's because Fd() returns uintptr which is unsigned and can't be negative. IOW, a File object should always be a valid opened fd.

rhatdan commented 3 weeks ago

/approve

openshift-ci[bot] commented 3 weeks ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, giuseppe, rhatdan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/containers/storage/blob/main/OWNERS)~~ [giuseppe,rhatdan] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment