Skip to content

Follow-up: Exercise GitHub Actions changes from PR #871 #872

Closed

Description

@justin808

🤖 Codex · OpenAI · gpt-6.1-sol · medium

Source PR: #871

Exercise the Ruby 3.4.9 workflow inputs after the PR merges. This tracks verification of the changed GitHub Actions behavior.

Changed workflows:

  • .github/workflows/lint_test.yml
  • .github/workflows/js_test.yml
  • .github/workflows/rspec_test.yml
  • .github/workflows/demo-fleet-smoke.yml

Post-merge event: the merge commit's push to master must trigger Lint CI, JS CI, Rspec CI, and Demo fleet smoke.

Expected evidence:

  • Record the merge commit and exact-head run URLs for all four workflows.
  • Confirm Ruby setup selects 3.4.9 successfully, dependency installs are frozen where declared, and the workflows reach their lint/test/build/smoke stages.
  • Confirm RSpec asset prebuild and runtime both use test mode, including a cold-cache SSR/RSC system-test pass without changing the artifact consistency guard.
  • Confirm all applicable jobs pass; preserve and classify any new failure instead of treating Ruby setup success as full validation.
  • Retain the fresh-head PR review-app deployment and smoke evidence separately; runtime version detection must agree with the merged .ruby-version.

Cleanup: no verification-only PR is planned. If one becomes necessary, close it after collecting evidence, delete its branch, and remove its review environment with the repository's documented review-app cleanup command. Do not deploy production for this exercise.

Owner: release coordinator / maintainer.

Close this issue only after the exact merge-head evidence is recorded and any remaining failure has an explicit disposition.

Activity

  1. justin808 commented on Oct 27, 2026

    @justin808
    MemberAuthor

    🤖 Codex · OpenAI · gpt-6.1-sol (configured) · medium

    Verified the push event from merge commit 14cd65d. All four changed workflows passed on that exact commit:

    The logs confirm Ruby 3.4.9 setup and actual install/build/test execution. RSpec asset prebuild and renderer/spec execution inherit NODE_ENV=test; the artifact consistency guard passed. The retained pre-merge cold-cache control reproduced the old mixed-mode failure and passed with this fix.

    The fetched default branch contains the merge, whose tree exactly matches the independently audited PR head. Actual PR preview build, deployment and browser SSR/RSC interactions passed before merge. Automatic PR-close cleanup passed; the assigned preview and image are absent, its secret-policy binding is removed, and shared resources remain. No verification-only PR was needed.

    Verdict: passed, tester Codex, 2026-10-27 UTC. The requested workflow exercise is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions