Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix gcsfs RemoveAll #340

merged 4 commits into from Mar 11, 2022


Copy link

@typerat typerat commented Feb 22, 2022

RemoveAll in gcsfs returns an error when the file/directory to be removed doesn't exist. It should return nil in this case.

Copy link

CLAassistant commented Feb 22, 2022

CLA assistant check
All committers have signed the CLA.

Copy link

mbertschler commented Mar 11, 2022

Thanks for your first contribution @typerat

@mbertschler mbertschler merged commit 931302a into spf13:master Mar 11, 2022
2 checks passed
kodiakhq bot pushed a commit to cloudquery/cloudquery that referenced this pull request Jun 27, 2022

This PR contains the following updates:

| Package | Type | Update | Change |
| []( | require | minor | `v1.7.0` -> `v1.8.2` |


### Release Notes


### [`v1.8.2`](

[Compare Source](

-   Fix gcsfs RemoveAll by [@&#8203;typerat]( in [spf13/afero#340
-   bugfix: RemoveAll by [@&#8203;yearnfar]( in [spf13/afero#329

### [`v1.8.1`](

[Compare Source](

### [`v1.8.0`](

[Compare Source](

-   sftpfs: Add support for Readdirnames and Readdir
-   CacheOnReadFs: Call OpenFile instead of Open in CacheOnReadFs.OpenFile

### [`v1.7.1`](

[Compare Source](



### Configuration

📅 **Schedule**: Branch creation - "before 3am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

 **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.


 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.


This PR has been generated by [Renovate Bot](
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

3 participants