Skip to content

Conversation

philprime
Copy link
Member

No description provided.

@Copilot Copilot AI review requested due to automatic review settings September 29, 2025 11:22
@philprime philprime enabled auto-merge (squash) September 29, 2025 11:23
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors Sentry configuration to use environment-specific settings and updates dependencies. The main changes include creating a helper function to select different configuration values based on build environment (DEBUG vs production) and updating the Sentry SDK from version 8.56.0-alpha.3 to 8.56.2.

  • Added pickEnvValue helper function for environment-based configuration selection
  • Updated Sentry sample rates to use higher values in development (1.0) vs production (0.1-0.2)
  • Updated Sentry SDK version from alpha to stable release

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
Targets/App/Sources/Main/FlinkyApp.swift Added environment-based configuration helper and updated Sentry sampling rates
Targets/App/Sources/Resources/Settings.bundle/Licenses.plist Updated Sentry version reference in licenses
Targets/App/Sources/Resources/Settings.bundle/Licenses.latest_result.txt Updated license tracking for new Sentry version
.swiftlint.yml Added .git directory to excluded paths

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@philprime philprime merged commit 14611f9 into main Sep 29, 2025
3 checks passed
@philprime philprime deleted the env-helper branch September 29, 2025 11:34
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