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(github): fetchFiles should return files #229

Merged
merged 1 commit into from
May 24, 2023
Merged

fix(github): fetchFiles should return files #229

merged 1 commit into from
May 24, 2023

Conversation

and-cesbo
Copy link
Contributor

fixed lost return in fetchFiles function

@cpreston321
Copy link
Member

Resolves: #223

@Barbapapazes
Copy link
Member

Nice! Thanks! 👍

Copy link
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice spot thanks!

@pi0 pi0 merged commit 7afac4c into unjs:main May 24, 2023
so1ve pushed a commit to so1ve/unstorage that referenced this pull request Jul 8, 2023
* fix(prefixStorage): prefix `getItemRaw` and `setItemRaw` (unjs#232)

* fix(github): fetchFiles should return files (unjs#229)

* chore(deps): update all non-major dependencies (unjs#220)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: update eslint

* test: skip cloudflare-kv-http on node >= 18

* chore(release): v1.6.1

* docs: add social share image

* chore: update deps

* docs: fix typo (unjs#239)

Change `environemnt` to `environment`
Update cloudflare-kv-http.md

* chore: update dependencies

* feat: generic type support (unjs#237)

* refactor: fix issues with typescript strict (unjs#250)

* chore: add type check to ci

* ci: skip flaky azure tests

* chore(release): v1.7.0

* chore(deps): update all non-major dependencies

* docs: fix typo (unjs#252)

* chore(deps): update all non-major dependencies

* test: add test for `github` driver (unjs#259)

* feat: experimental operation batching (unjs#240)

Co-authored-by: Pooya Parsa <pooya@pi0.io>

* feat(cloudflare-kv): support `base` option for keys (unjs#261)

* feat: `cloudflare-r2-binding` driver (unjs#235)

* fix: add missing `cloudflareR2Binding` to the `builtinDrivers`

* chore: update dev dependencies

* chore(release): v1.8.0

* Fix typescript checks

* add typehint

* Install execa

* Write test code

* Remove not using import

---------

Co-authored-by: 魔王少年 <q267009886.work@gmail.com>
Co-authored-by: Andrei Dyldin <and@cesbo.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Pooya Parsa <pooya@pi0.io>
Co-authored-by: Sébastien Chopin <seb@nuxt.com>
Co-authored-by: Neelansh Mathur <53081208+neelansh15@users.noreply.github.com>
Co-authored-by: 魔王少年 <q267009886.tw@gmail.com>
Co-authored-by: Alex Duval <alexduval71@gmail.com>
Co-authored-by: Hebilicious <xsh4k3@gmail.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.

4 participants