Skip to content

Release/pipeline quality gates - #10

Merged
dchuk merged 13 commits into
mainfrom
release/pipeline-quality-gates
Oct 22, 2025
Merged

dchuk merged 13 commits into
mainfrom
release/pipeline-quality-gates

Conversation

@dchuk

@dchuk dchuk commented Oct 21, 2025

Copy link
Copy Markdown
Owner

automation and CI enhancements in preparation for releasing the engine as a gem

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/support/host_app_harness.rb Outdated
@dchuk

dchuk commented Oct 22, 2025

Copy link
Copy Markdown
Owner Author

@codex how do I resolve this pipeline failures

@dchuk
dchuk marked this pull request as draft October 22, 2025 01:59
@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

  • Updated the host app harness SQLite initializer to reuse a shared shim loader, hook into the :active_record_sqlite3adapter lazy-load event, and require the adapter eagerly so the jsonb alias is applied whenever SQLite is in use. test/support/host_app_harness.rbL301-L344

Testing

  • ⚠️ Not run (not requested)

View task →

@dchuk
dchuk marked this pull request as ready for review October 22, 2025 03:15
@dchuk
dchuk merged commit 3bbda92 into main Oct 22, 2025
4 checks passed
@dchuk
dchuk deleted the release/pipeline-quality-gates branch February 13, 2026 16:52
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.

1 participant