Skip to content
This repository was archived by the owner on Sep 13, 2023. It is now read-only.

remove recursive flag for ls #393

Merged
merged 1 commit into from
Sep 5, 2022
Merged

remove recursive flag for ls #393

merged 1 commit into from
Sep 5, 2022

Conversation

mike0sv
Copy link
Contributor

@mike0sv mike0sv commented Sep 1, 2022

closes #384

@mike0sv mike0sv requested a review from a team September 1, 2022 21:31
@mike0sv mike0sv self-assigned this Sep 1, 2022
@mike0sv mike0sv temporarily deployed to internal September 1, 2022 21:31 Inactive
@codecov
Copy link

codecov bot commented Sep 1, 2022

Codecov Report

Base: 89.51% // Head: 89.51% // No change to project coverage 👍

Coverage data is based on head (c81b3ea) compared to base (8a409c8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #393   +/-   ##
=======================================
  Coverage   89.51%   89.51%           
=======================================
  Files          81       81           
  Lines        6276     6276           
=======================================
  Hits         5618     5618           
  Misses        658      658           
Impacted Files Coverage Δ
mlem/core/index.py 65.71% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@aguschin aguschin merged commit 43026ea into main Sep 5, 2022
@aguschin aguschin deleted the fix/ls-s3 branch September 5, 2022 06:43
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Issue listing MLEM objects on S3
2 participants