-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
chore(flutter): Add missing options to download sentry-wizard for getting started page #14120
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
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
👀 will test it tmrw |
Bundle ReportChanges will increase total bundle size by 126 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
# Dynamic Sentry Wizard Version Implementation | ||
|
||
## Current State | ||
The Flutter and Android documentation now includes comprehensive installation options for sentry-wizard, but the version is currently hardcoded to v5.1.0 (updated from the previous v4.0.1). | ||
|
||
## Problem Solved | ||
- ✅ Added comprehensive platform support (Windows, macOS Intel/ARM, Linux x64/ARM) | ||
- ✅ Updated to latest version (v5.1.0 from v4.0.1) | ||
- ✅ Windows users no longer forced to use `brew` | ||
- ✅ Flutter developers can avoid JavaScript tooling (`npx`) if preferred | ||
- ✅ Consistent with Android documentation approach | ||
|
||
## Future Enhancement: Dynamic Version Fetching | ||
|
||
### Approach 1: Build-time Version Fetching (Recommended) | ||
Create a build script that fetches the latest version during the build process: | ||
|
||
```javascript | ||
// scripts/fetch-sentry-wizard-version.js | ||
const fs = require('fs'); | ||
const path = require('path'); | ||
|
||
async function fetchLatestVersion() { | ||
try { | ||
const response = await fetch('https://api.github.com/repos/getsentry/sentry-wizard/releases/latest'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this file intended?
```bash {tabTitle:macOS (Intel/x64)} | ||
downloadUrl="https://github.com/getsentry/sentry-wizard/releases/download/v5.1.0/sentry-wizard-darwin-x64" | ||
curl -L $downloadUrl -o sentry-wizard | ||
chmod +x sentry-wizard | ||
./sentry-wizard -i flutter | ||
``` | ||
|
||
```bash {tabTitle:macOS (Apple Silicon/arm64)} | ||
downloadUrl="https://github.com/getsentry/sentry-wizard/releases/download/v5.1.0/sentry-wizard-darwin-arm64" | ||
curl -L $downloadUrl -o sentry-wizard | ||
chmod +x sentry-wizard | ||
./sentry-wizard -i flutter | ||
``` | ||
|
||
```bash {tabTitle:Linux (x64)} | ||
downloadUrl="https://github.com/getsentry/sentry-wizard/releases/download/v5.1.0/sentry-wizard-linux-x64" | ||
curl -L $downloadUrl -o sentry-wizard | ||
chmod +x sentry-wizard | ||
./sentry-wizard -i flutter | ||
``` | ||
|
||
```bash {tabTitle:Linux (arm64)} | ||
downloadUrl="https://github.com/getsentry/sentry-wizard/releases/download/v5.1.0/sentry-wizard-linux-arm64" | ||
curl -L $downloadUrl -o sentry-wizard | ||
chmod +x sentry-wizard | ||
./sentry-wizard -i flutter | ||
``` | ||
|
||
```powershell {tabTitle:Windows} | ||
$downloadUrl = "https://github.com/getsentry/sentry-wizard/releases/download/v5.1.0/sentry-wizard-win-x64.exe" | ||
Invoke-WebRequest $downloadUrl -OutFile sentry-wizard.exe | ||
./sentry-wizard.exe -i flutter | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's switch order of Windows and NPX (so that Windows is the 2nd)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a little confused by this root-level .md file, is this intended specifically for editors who have pulled down the docs repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR was created by Cursor through slack so I think that was also added there unintentionally
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @romtsn
The Flutter onboarding documentation in
docs/platforms/dart/guides/flutter/index.mdx
was updated to align with Android'ssentry-wizard
installation methods. Previously, onlybrew
andnpx
options were available, which limited support for Windows users and forced the use of JavaScript tooling.Changes include:
sentry-wizard
installation methods: macOS (Intel/x64, Apple Silicon/arm64), Linux (x64, arm64), and Windows (PowerShell). This provides direct binary downloads, addressing platform compatibility.These updates ensure comprehensive platform coverage and offer native tooling alternatives, improving accessibility for Flutter developers across various operating systems.