Skip to content

fix: bulk update regression duplicates - #8275

Merged
mergify[bot] merged 2 commits into
mainfrom
cal/fix-content-duplication-issue-prepare
Nov 10, 2026
Merged

mergify[bot] merged 2 commits into
mainfrom
cal/fix-content-duplication-issue-prepare

Conversation

@IMB11

@IMB11 IMB11 commented Nov 9, 2026

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

  • Fixed issue where doing a bulk update could break if a mod being updated now depended on a project which was already installed as an external file.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@greptile-apps

greptile-apps Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; the previous issues are addressed and no new blocking issue was found.

Summary

This PR fixes how bulk updates merge duplicate copies.

  • Bulk updates replace duplicate project copies with one updated copy.

All three previous findings are addressed. No new actionable issues or repository-rule violations were found. This review used source inspection; no runtime tests were run.

Reviews (2) · Last reviewed commit: "fix: rev" · Reviewed by Greptile

@IMB11

IMB11 commented Nov 9, 2026

Copy link
Copy Markdown
Member Author

@greptileai review

@mergify

mergify Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Queue requested — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

✅ Merged into main as 68a23cb at 15:44 UTC

1 min in the queue, with no time running CI.

So far

2 merge conditions met
  • github-review-approved · required by GitHub branch protection
  • any of · required by GitHub branch protection:
    • check-skipped = Lint and Test
    • check-neutral = Lint and Test
    • check-success = Lint and Test

Times in UTC · dashboard · how the merge queue works

@mergify mergify Bot added the queued label Nov 10, 2026
@mergify
mergify Bot merged commit 68a23cb into main Nov 10, 2026
8 checks passed
@mergify
mergify Bot deleted the cal/fix-content-duplication-issue-prepare branch November 10, 2026 15:44
@mergify mergify Bot removed the queued label Nov 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants