Skip to content

test: preview-screenshot check (transient — will close) - #668

Closed
JSONbored wants to merge 1 commit into
mainfrom
test/reviewbot-preview-check
Closed

test: preview-screenshot check (transient — will close)#668
JSONbored wants to merge 1 commit into
mainfrom
test/reviewbot-preview-check

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Verifying the before/after preview screenshots work end-to-end after the CI fixes (#653/#654/#658). Changes the homepage hero (Not like a bot.Not like a bot. (preview check)) so the after screenshot should differ from before. Transient; will close.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 14, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
gittensory-ui 996fe69 Commit Preview URL

Branch Preview URL
Jun 14 2026, 01:16 AM

@ghost

ghost commented Jun 14, 2026

Copy link
Copy Markdown

Note

Gittensory Gate skipped

PR closed before full evaluation. No late first comment was created.

Signal Result Evidence Action
Gate result ⚠️ Skipped #668 is no longer open. No action.

💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.

@ghost ghost added the gittensory:reviewed label Jun 14, 2026
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@ghost

ghost commented Jun 14, 2026

Copy link
Copy Markdown

gittensory · advisory review

Reviewed 1 changed file(s) — two independent AI reviewers.

Suggested action:Don't merge — close. (reviewers split: merge / close) At least one reviewer recommends against this change; resolve it before reconsidering.

Reviewer A · gpt-oss-120b — recommends ✅ merge
This PR adds a '(preview check)' label to the hero subtitle in the UI, a minor text change with no functional impact.

Suggestions

  • Consider removing the '(preview check)' annotation before the final release if it is only for testing.
  • Run the UI test suite to ensure the change doesn't affect any string‑matching logic.

Reviewer B · nemotron-3-120b-a12b — recommends ⛔ close
The PR adds a transient test string '(preview check)' to the hero text on the index page. The change is trivial and purely textual, but the PR title explicitly states it is a test and will be closed, indicating the author does not intend for it to be merged.

No blocking issues spotted.

Before = production · After = this PR's preview deploy.

Route Viewport Before After
/ desktop before desktop after desktop
mobile before mobile after mobile

@ghost ghost added the gittensory-review label Jun 14, 2026
@JSONbored JSONbored closed this Jun 14, 2026
@JSONbored
JSONbored deleted the test/reviewbot-preview-check branch June 14, 2026 01:19
@github-project-automation github-project-automation Bot moved this from Todo to Done in gittensory - v1 roadmap Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant