Closed milescrabill closed 3 months ago
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
@peczenyj it seems you've had some luck getting things merged into this repo. Do you have any tips or tricks on how to get this one reviewed?
Please be patient.
I had no idea, to be honest. I send the first pull request without any expectations based on my previous attempts to contribute to other google projects. Somehow this project is different, maybe it is more close to the open source community.
Is there any mailing list where you can expose your problem and mention that you have an awesome pull request for that?
If the maintainers do not react with the necessary speed, perhaps you should consider fork the project - if the license allow - and if your fork get some attention maybe it change things (like archive this repo and point yours as the replacement - why not?)
Sometimes I just start pull requests because code can be more direct than discussions. But not all maintainers like this approach, since I can suggest something undesirable or using the wrong design. It can end in frustration. But I try not to be sentimental.
When issues or pull requests have no feedback for some time it give the wrong impression that they just don’t care. But in fact they can be very busy.
In fact I saw, on this project, many small opportunities to contribute and things to improve, etc. I would love continue to send PRs but first I’d to see what I just did being released.
maybe one day I can have a more substantial role, the I can do more than just write random thoughts
Or we can just ping @vangent and try to bribe him and with some beers
Attention: Patch coverage is 0%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 73.37%. Comparing base (
ba58ec7
) to head (0e27588
).
Files | Patch % | Lines |
---|---|---|
blob/blob_fs.go | 0.00% | 6 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Motivation:
http.ServeFileFS()
useserrors.Is()
to match onfs.ErrNotExist
andfs.ErrPermission
errors and return corresponding HTTP status codes.fs.PathError
which do not wrap the underlyingfs
errors, and which are not handled byhttp.ServeFileFS()
's error handling.fs.PathError
s results inhttp.ServeFileFS()
serving Internal Server Errors instead of Not Found or Permission Denied errors.Fix:
Open()
around theirio/fs
counterparts