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

Dependencies #380

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Dependencies #380

wants to merge 28 commits into from

Conversation

KKA11010
Copy link
Collaborator

@KKA11010 KKA11010 commented Feb 20, 2025

This pull request includes several major changes across multiple files, focusing on updating dependencies, refactoring environment variable usage, expo-sqlite usage and reconfiguring ESLint settings.

Dependency Updates:

  • Updated various dependencies in package.json to their latest versions, including expo, react-native, @sentry/react-native, and the changes from expo-sqlite/legacy to expo-sqlite. And many other dependencies.

Environment Variable Refactoring:

  • Replaced direct usage of process.env with a constant ENV in config/app.config.ts to streamline environment variable handling. [1] [2]

ESLint Configuration:

  • Removed the old .eslintrc.json configuration file and replaced it with a new eslint.config.mjs file, which includes updated ESLint rules and plugins. [1] [2]
  • Simplified the ESLint command in package.json to exclude file extensions.

Code Refactoring:

  • Enabled new architecture in config/app.config.ts and added configuration for expo-sqlite with specific settings for Android and iOS. [1] [2]
  • Wrapped the App component with Sentry's withProfiler in src/App.ts to enable performance monitoring.

Migrating Tests

  • Tests are currently failing due to major changes in the codebase.
  • Sentry has configuration issues and needs further investigation
  • E2E Tests using Detox are working well exept of the debug-build
  • Tests fixed -- edit: Nodejs Tests are currently ignored and replaced by slower E2E tests.
  • Sentry fixed
  • Detox fixed

@KKA11010 KKA11010 marked this pull request as ready for review February 22, 2025 15:27
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.

1 participant