Skip to content

Ignore transactional install state#908

Merged
ilysenko merged 1 commit into
ilysenko:mainfrom
yazanabuashour:codex/ignore-transactional-build-state
Jul 12, 2026
Merged

Ignore transactional install state#908
ilysenko merged 1 commit into
ilysenko:mainfrom
yazanabuashour:codex/ignore-transactional-build-state

Conversation

@yazanabuashour

@yazanabuashour yazanabuashour commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • ignore transactional candidate directories, recovery journals, and promotion locks
  • prevent normal local builds from dirtying the checkout

Follow-up to #834.

Validation

  • git diff --check
  • verified all three paths with git check-ignore

Checklist

  • This pull request is ready for review and is no longer a draft.
  • I followed CONTRIBUTING.md, kept the change focused, edited source files rather than generated output, and removed unrelated changes.
  • If this fixes upstream drift, it targets only the latest CODEX.DMG and removes obsolete fallback code and tests from the affected area.
  • I added or updated relevant tests, ran the validation listed above, and confirmed that required CI checks pass.
  • I reviewed the final diff with my coding agent using maximum reasoning effort, addressed all findings, and reran the relevant tests.

@ilysenko ilysenko left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Reviewed the current head. The ignore rules are narrowly scoped to generated transactional install state, match the documented runtime paths, and all required checks pass.

@ilysenko
ilysenko merged commit 6468ecf into ilysenko:main Jul 12, 2026
6 checks passed
@yazanabuashour
yazanabuashour deleted the codex/ignore-transactional-build-state branch July 12, 2026 18:21
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