Skip to content

Conversation

naps62
Copy link
Member

@naps62 naps62 commented Jul 29, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 29, 2025 20:58
@naps62 naps62 added the B-bugfix Something is working again label Jul 29, 2025
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 enables Aptabase analytics on release builds by configuring the necessary environment variables and build features for production releases.

  • Adds APTABASE_API secret to the release workflow environment
  • Enables the "production" feature flag for release builds

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/release.yml Adds APTABASE_API environment variable and production feature flag to release builds
result Removes a Nix store path reference (likely build artifact cleanup)

@@ -182,6 +182,8 @@ jobs:
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

APTABASE_API: ${{ secrets.APTABASE_API }}
Copy link
Preview

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

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

The APTABASE_API secret should be validated to ensure it exists before use. Consider adding a conditional check or using a default value to prevent build failures if the secret is not configured.

Copilot uses AI. Check for mistakes.

@naps62 naps62 enabled auto-merge (squash) July 30, 2025 10:47
@naps62 naps62 merged commit d629d3b into main Jul 30, 2025
7 checks passed
@naps62 naps62 deleted the aptabase branch July 30, 2025 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-bugfix Something is working again
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant