ScoopInstaller / Main

📦 The default bucket for Scoop.
https://scoop.sh
The Unlicense
1.58k stars 958 forks source link

Add pocketbase@0.22.10 #5749

Closed simonsan closed 4 months ago

simonsan commented 5 months ago

Closes #5748

github-actions[bot] commented 5 months ago

Your changes do not pass checks.

Invalid manifests

simonsan commented 5 months ago

/verify

github-actions[bot] commented 5 months ago

All changes look good.

Wait for review from human collaborators.

pocketbase

aliesbelik commented 5 months ago

As from documentation:

The prebuilt PocketBase executable will automatically create and manage 2 new directories alongside the executable:

  • pb_data - stores your application data, uploaded files, etc. (usually should be added in .gitignore).
  • pb_migrations - contains JS migration files with your collection changes (can be safely committed in your repository).

Wonder if at least pb_data is worth to be persistent - to store between updates.

simonsan commented 5 months ago

As from documentation:

The prebuilt PocketBase executable will automatically create and manage 2 new directories alongside the executable:

  • pb_data - stores your application data, uploaded files, etc. (usually should be added in .gitignore).
  • pb_migrations - contains JS migration files with your collection changes (can be safely committed in your repository).

Wonder if at least pb_data is worth to be persistent - to store between updates.

Yeah, probably both are worth to be persisted?

simonsan commented 5 months ago

/verify

github-actions[bot] commented 5 months ago

All changes look good.

Wait for review from human collaborators.

pocketbase

simonsan commented 5 months ago

@aliesbelik Want to take another look, if this is enough?

aliesbelik commented 5 months ago

Looks good to me. But since i don't have write access in Main, please wait a bit more for merging.

simonsan commented 5 months ago

Looks good to me. But since i don't have write access in Main, please wait a bit more for merging.

Yes, I'm aware. Was really just to make sure, some future person merging finds a good to go PR. :) Thanks for looking over it. 👍🏽

tech189 commented 4 months ago

Thanks @simonsan and @aliesbelik!