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

feat(sqlite-storage): scaffold workspace #1071

Merged
merged 15 commits into from
Apr 8, 2024

Conversation

krizzu
Copy link
Member

@krizzu krizzu commented Mar 14, 2024

Summary

Beginning to work on SQLite storage. Basic scaffold for native module + example app using RN test app.

packages/sqlite-storage/.gitignore Outdated Show resolved Hide resolved
packages/sqlite-storage/RNAsyncStorageSQLite.podspec Outdated Show resolved Hide resolved
packages/sqlite-storage/RNAsyncStorageSQLite.podspec Outdated Show resolved Hide resolved
packages/sqlite-storage/example/android/build.gradle Outdated Show resolved Hide resolved
packages/sqlite-storage/example/android/settings.gradle Outdated Show resolved Hide resolved
packages/sqlite-storage/example/ios/Podfile Outdated Show resolved Hide resolved
packages/sqlite-storage/example/ios/Podfile Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
packages/sqlite-storage/tsconfig.json Outdated Show resolved Hide resolved
* explicit prettierrc

* turbo

* format test

* pull-request workflow

* update actions

* bump to yarn 4

* build script
@krizzu krizzu force-pushed the sqlite/scaffold-workspace branch from 0cab7d7 to 322c03a Compare April 8, 2024 09:44
@@ -18,7 +18,7 @@ jobs:
token: ${{ secrets.GH_RELEASE_TOKEN }}
filters: |
storage:
- 'packages/default-storage'
- 'packages/default-storage/**'
Copy link
Member Author

Choose a reason for hiding this comment

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

small bonus 🙏

@@ -5,9 +5,18 @@ inputs:
required: false
description: node version to install
default: 20.10.0 # is already cached in Ubuntu 22.04 runner
windows-fix:
required: false
description: https://github.com/actions/setup-node/issues/899#issuecomment-1837381044
Copy link
Member

Choose a reason for hiding this comment

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

If corepack randomly fails on Windows, maybe we shouldn't enable it?

Copy link
Member Author

Choose a reason for hiding this comment

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

GH action failed because it was not enabled in first place - that issue came from setup-node itself. Running it first without the cache enables corepack properly (probably not doing checks for yarn versions behind the scene). So the fix here is simply setting node + corepack then enable cache

@krizzu
Copy link
Member Author

krizzu commented Apr 8, 2024

Thanks for review @tido64 ❤️

@krizzu krizzu merged commit 7621aaf into sqlite-storage Apr 8, 2024
9 checks passed
@krizzu krizzu deleted the sqlite/scaffold-workspace branch April 8, 2024 15:11
krizzu added a commit that referenced this pull request Apr 8, 2024
* chore(deps): bump follow-redirects from 1.15.4 to 1.15.6 (#1074)

* feat(default-storage): Add Privacy Manifest for iOS (#1075)

privacy file

* chore(release): 1.23.0 [skip ci]

* fix(default-storage): Privacy manifest missing key (#1076)

add missing key

* chore(release): 1.23.1 [skip ci]

* chore(deps): bump webpack-dev-middleware from 5.3.3 to 5.3.4 (#1077)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 5.3.3 to 5.3.4.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v5.3.4/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v5.3.3...v5.3.4)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: GH workflows, turbo and other chores (#1073)

* explicit prettierrc

* turbo

* format test

* pull-request workflow

* update actions

* bump to yarn 4

* build script

* init

* examples

* comments

* fix glob

* namespace

* windows build

* fixing windows action

* android example search for node_modules

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: AsyncStorageBot <118817110+AsyncStorageBot@users.noreply.github.com>
krizzu added a commit that referenced this pull request Apr 10, 2024
sqlite-storage: scaffold workspace (#1071)

* chore(deps): bump follow-redirects from 1.15.4 to 1.15.6 (#1074)

* feat(default-storage): Add Privacy Manifest for iOS (#1075)

privacy file

* chore(release): 1.23.0 [skip ci]

* fix(default-storage): Privacy manifest missing key (#1076)

add missing key

* chore(release): 1.23.1 [skip ci]

* chore(deps): bump webpack-dev-middleware from 5.3.3 to 5.3.4 (#1077)

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) from 5.3.3 to 5.3.4.
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v5.3.4/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v5.3.3...v5.3.4)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
...




* chore: GH workflows, turbo and other chores (#1073)

* explicit prettierrc

* turbo

* format test

* pull-request workflow

* update actions

* bump to yarn 4

* build script

* init

* examples

* comments

* fix glob

* namespace

* windows build

* fixing windows action

* android example search for node_modules

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: AsyncStorageBot <118817110+AsyncStorageBot@users.noreply.github.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.

3 participants