Skip to content

Improve native-dependency-update skill instructions#3477

Merged
mattleibow merged 1 commit intomainfrom
dev/improve-native-dep-skill
Jan 29, 2026
Merged

Improve native-dependency-update skill instructions#3477
mattleibow merged 1 commit intomainfrom
dev/improve-native-dep-skill

Conversation

@mattleibow
Copy link
Contributor

Summary

Improves Phase 5 (Create PRs) of the native-dependency-update skill to prevent common mistakes.

Changes

  • Add branch naming convention table for quick reference
  • Add numbered sub-steps for SkiaSharp PR creation
  • Add ⚠️ CRITICAL callout for submodule update requirement
  • Add Phase 5 completion checklist to verify all steps before proceeding
  • Use natural language instead of rigid bash scripts for flexibility

Why

During a libwebp update, I made these mistakes:

  1. Used wrong branch naming (libwebp-1.6.0 instead of dev/update-libwebp)
  2. Forgot to update the externals/skia submodule in the SkiaSharp PR
  3. Didn't cross-link the PRs

The previous instructions had these issues buried in prose, making them easy to miss when moving quickly.

- Add branch naming convention table for quick reference
- Add numbered sub-steps for SkiaSharp PR creation
- Add CRITICAL callout for submodule update requirement
- Add Phase 5 completion checklist
- Use natural language instead of rigid bash scripts
@github-actions
Copy link

Triage Summary

Labels will be applied to indicate the area of improvement (Documentation) and the relevant backend library (SkiaSharp) involved in the issue.

This issue is not a regression and does not involve compatibility, performance, or reliability issues.

Additional remarks:

  • The issue centers on enhancing documentation to prevent mistakes during the native-dependency-update skill process, without direct ties to specific operating systems or performance concerns.
Detailed Summary and Actions

Summary of the triage:

  • The issue focuses on improving documentation for the native-dependency-update skill, particularly to enhance instructions.
  • Relevant improvements are noted for the SkiaSharp library in the context of PR creation.
  • There are no operating system or performance labels applicable, indicating that the issue is not related to system compatibility or reliability.

Summary of the actions that will be performed:

Action Item Description
Apply Label area/Docs The issue involves improving documentation for the native-dependency-update skill.
Apply Label backend/SkiaSharp Improvements mentioned in relation to SkiaSharp which pertains specifically to PR creation.

This entire triage process was automated by AI and mistakes may have been made. Please let us know so we can continue to improve.

@github-actions github-actions bot added area/Docs Issues relating to documentation, such as API docs or conceptual docs. backend/SkiaSharp labels Jan 29, 2026
@mattleibow mattleibow merged commit 6f1ee5b into main Jan 29, 2026
2 checks passed
@mattleibow mattleibow deleted the dev/improve-native-dep-skill branch January 29, 2026 18:59
@mattleibow mattleibow added the copilot Created by GitHub Copilot label Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/Docs Issues relating to documentation, such as API docs or conceptual docs. copilot Created by GitHub Copilot

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant