preparing-pr

Prepare a pull request by running presubmit with fixes. Use when preparing to commit or send a PR.

npx skills add https://github.com/flutter/devtools --skill preparing-pr

Preparing a Pull Request

Follow these steps to prepare a pull request for DevTools:

  1. Verify dt configuration:
    • Ensure dt is available on the path by running dt --help or verifying its alias.
    • If not available, refer to the CONTRIBUTING guide for instructions on how to set up the dt executable on your path.
  2. Update Flutter SDK:
    • Run dt update-flutter-sdk to ensure the SDK is up to date.
  3. Run Presubmit with Fixes:
    • Run dt presubmit --fix to address automated fixes.
    • If any issues remain or if the command fails, fix the issues manually and run dt presubmit --fix again to verify the fix.
  4. Run Tests for Affected Code:
    • Use git status or git diff upstream/master to find changed files.
    • Run tests that are associated with the changed code. For example, if you edited packages/devtools_app/lib/src/screens/logging/logging_screen.dart, run the logging tests under packages/devtools_app/test/screens/logging/.

Verification

If any step fails, stop and address the issue, and then verify the issue is fixed before proceeding.

[!IMPORTANT] Do NOT commit or push changes without getting explicit user approval first.

More skills from flutter

dart-modern-features
flutter
To find candidates for modernization:
flutter-fix-layout-issues
flutter
Fixes Flutter layout errors (overflows, unbounded constraints) using Dart and Flutter MCP tools. Use when addressing "RenderFlex overflowed", "Vertical…
adding-release-notes
flutter
Adds user-facing change descriptions to DevTools release notes. Use when documenting improvements, fixes, or new features in the NEXT_RELEASE_NOTES.md file.
reviewing-devtools-prs
flutter
DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns. Use when reviewing pull requests in the…
dart-use-primary-constructors
flutter
Help users write syntactically and semantically correct primary constructors in Dart, and migrate/use the new constructor syntax, empty-body semicolon syntax,…
code-documentation
flutter
Guide for writing effective code documentation, including docstrings, JSDoc, dartdoc, and implementation comments. Use this skill when writing new code, adding…
api-review
flutter
Reviews the specified code against the canonical API Design guidelines. Use this skill when the user asks for an API review or to check code against API design…
flutter-accessibility
flutter
Implement WCAG 2 and EN 301 549 accessibility standards and adaptive layouts in Flutter apps. Enforces semantic annotations, tap target sizes (48x48 dp minimum), and text contrast ratios (4.5:1 for small text, 3:1 for large) across mobile, web, and desktop platforms Provides decision logic for web semantics initialization, interactive widget wrapping, screen-size-based layout switching, and keyboard/mouse input handling Includes focus traversal management via FocusTraversalGroup and...