Skip to content

Promote Develop to Main - #70

Merged
ptr727 merged 2 commits into
mainfrom
develop
Aug 18, 2026
Merged

Promote Develop to Main#70
ptr727 merged 2 commits into
mainfrom
develop

Conversation

@ptr727

@ptr727 ptr727 commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • promote the current develop branch to main
  • release the carried fleet skill updates from Sync Carried Fleet Skills #69
  • make fresh PhotoCleaner worktrees prepare their Husky.Net runtime before the first commit

Verification

  • Sync Carried Fleet Skills #69 passed its full local build, test, hook, and lint verification
  • the final Sync Carried Fleet Skills #69 head passed all 17 required checks
  • Copilot reviewed the full final diff with no current-head findings
  • every finding from earlier review rounds was fixed and explicitly answered

ptr727 added 2 commits August 18, 2026 11:32
* Sync Carried Fleet Skills

* Clarify Carried Review Helper Location

* Combine Carried Review Wait Steps
Copilot AI lite review requested due to automatic review settings August 18, 2026 20:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Promotes the current develop state to main, including a small Import deduplication correctness fix plus carried fleet skill updates that improve new-worktree setup and review/verification guidance.

Changes:

  • Update ImportTask to update an existing source-path record (instead of INSERT OR IGNORE) when the source file is replaced with different content.
  • Add a regression test covering the “replaced source path” scenario to ensure the DB row is updated and subsequent runs skip as expected.
  • Refresh carried Skills documentation, including guidance to prepare Husky.Net / Python pre-commit hooks in new worktrees and clearer Python/PR-review workflow guidance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
PhotoCleaner/ImportTask.cs Updates Import.db write logic to update existing source-path rows when content changes.
PhotoCleanerTests/ImportTaskTests.cs Adds coverage for “replaced source” behavior and ensures a follow-up run skips without creating duplicate outputs.
.github/skills/repo-worktree/SKILL.md Adds explicit new-worktree guidance for preparing generated Git hook runtimes (Husky.Net / pre-commit).
.github/skills/python-codestyle/SKILL.md Clarifies Python verification guidance (read OPERATIONS.md first; avoid assuming pytest in lint-only repos).
.github/skills/pr-review-conduct/SKILL.md Tightens the PR-review loop contract to explicitly include PR opening, bounded waits, and hub-checkout requirements for tooling.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ptr727
ptr727 merged commit ca35d6a into main Aug 18, 2026
18 checks passed
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.

2 participants