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

test: for empty dir in ls #284

Merged
merged 1 commit into from
Jun 15, 2023
Merged

test: for empty dir in ls #284

merged 1 commit into from
Jun 15, 2023

Conversation

skshetry
Copy link
Member

No description provided.

@skshetry skshetry temporarily deployed to internal June 15, 2023 12:01 — with GitHub Actions Inactive
@skshetry skshetry requested review from efiop and daavoo June 15, 2023 12:01
@efiop
Copy link
Contributor

efiop commented Jun 15, 2023

welp, caught a bug 🙂

@efiop efiop merged commit c1a8428 into dvc-issue-9607 Jun 15, 2023
@efiop efiop deleted the test branch June 15, 2023 12:27
skshetry added a commit that referenced this pull request Jun 22, 2023
skshetry added a commit that referenced this pull request Jun 22, 2023
* GDriveFileSystem: Raise `FIleNotFoundError` on `ls`.

Per https://github.com/iterative/dvc/issues/9607

* Update pydrive2/fs/spec.py

Co-authored-by: Ruslan Kuprieiev <kupruser@gmail.com>

* black

* Test for empty dir in ls (#284)

* test_ls: create dir

mkdir in GDriveFileSystem is a noop.

---------

Co-authored-by: Ruslan Kuprieiev <kupruser@gmail.com>
Co-authored-by: skshetry <18718008+skshetry@users.noreply.github.com>
Co-authored-by: Saugat Pachhai (सौगात) <suagatchhetri@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants