Skip to content

chore: 上流のマージ時に削除済みファイルを維持するように修正#379

Merged
3w36zj6 merged 3 commits into
mainfrom
feature/auto-resolve-deleted-conflicts
Jun 23, 2026
Merged

chore: 上流のマージ時に削除済みファイルを維持するように修正#379
3w36zj6 merged 3 commits into
mainfrom
feature/auto-resolve-deleted-conflicts

Conversation

@3w36zj6

@3w36zj6 3w36zj6 commented Mar 12, 2026

Copy link
Copy Markdown
Member

上流リポジトリで削除されたファイルが意図せず残ることを防ぐためのマージ用のworkflowの修正です。

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

上流(typst/typst)を取り込むための merge-upstream ワークフローで、マージ時に「こちらで削除したファイル(deleted by us)」が上流側の変更で復活してしまうケースを防ぐための調整です。

Changes:

  • マージ後の未解決状態 DU(deleted by us)になっているファイルを検出し、git rm で削除として解決する処理を追加

Comment thread .github/workflows/merge-upstream.yml Outdated
@3w36zj6 3w36zj6 requested a review from Copilot June 19, 2026 11:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@3w36zj6 3w36zj6 force-pushed the feature/auto-resolve-deleted-conflicts branch from 0fdd432 to ea6e7cc Compare June 22, 2026 12:56
@3w36zj6 3w36zj6 marked this pull request as ready for review June 22, 2026 12:58
@ultimatile ultimatile requested a review from Copilot June 22, 2026 13:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread .github/workflows/merge-upstream.yml Outdated
Comment on lines +48 to +53
# Keep "deleted by us" files deleted
git status --porcelain=v1 -z | while IFS= read -r -d '' entry; do
case "$entry" in
"DU "*) git rm -- "${entry#DU }" ;;
esac
done
@3w36zj6 3w36zj6 requested a review from Copilot June 23, 2026 11:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment on lines +50 to +53
case "$entry" in
"DU "*) git rm -- "${entry#DU }" ;;
"UD "*) git rm -- "${entry#UD }" ;;
esac

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

@3w36zj6 3w36zj6 merged commit 432d7e1 into main Jun 23, 2026
5 checks passed
@3w36zj6 3w36zj6 deleted the feature/auto-resolve-deleted-conflicts branch June 23, 2026 12:41
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.

3 participants